-
Notifications
You must be signed in to change notification settings - Fork 21.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add URL for monitoring extension #37045
Conversation
@bragi92 : Thanks for your contribution! The author(s) have been notified to review your proposed change. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Recommend rewriting the statement to - Redirect URI: Two Web application types need to be created. The first base URL value should be https://afd.hosting.portal.azure.net/monitoring/Content/iframe/infrainsights.app/web/base-libs/auth/auth.html
and the second base URL value should be https://monitoring.hosting.portal.azure.net/monitoring/Content/iframe/infrainsights.app/web/base-libs/auth/auth.html
.
Thanks!
@bragi92 Would you make the recommended updates to the PR? Thanks. |
@bragi92 : Thanks for your contribution! The author(s) have been notified to review your proposed change. |
Updated. Thanks for the feedback. |
Thanks for making the change and it looks good. #sign-off |
No description provided.