Skip to content

Conversation

@tsultanov00
Copy link
Contributor

Description

  • Removed Business Service functionality from the UI
  • Added on-call input that creates a redirect link to Rootly OnCall
  • Cleaned up related test suite by removing Business Service tests

Contribution Checklist:

  • The pull request does not introduce any breaking changes
  • I have read the contribution guidelines.
  • Create an issue and link to the pull request.

Attach Screenshots (Optional)

Screenshot 2025-08-05 at 14 53 21 image Screenshot 2025-08-05 at 14 51 36

@tsultanov00 tsultanov00 force-pushed the rootly-oncall-id branch 2 times, most recently from 0138490 to 578e5da Compare August 11, 2025 17:10
const nextApp = next({ dev });
const debug = require('debug')('AthenzUI:server:app');

process.env.NEXT_PUBLIC_ONCALL_URL = appConfig.onCallUrl || appConfig.serverURL;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

apologies for the broken review, didn't notice it before
we do not have onCallUrl or serverURL in default-config.js.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Henry mentioned that we shouldn't expose internal on-call links so that was moved to the internal config. With regards to serverUrl, I could just change that to a generic fallback url i.e https://example.com

@havetisyan havetisyan merged commit 58a698c into AthenZ:master Aug 15, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants