Skip to content
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

Update sso-saml-technical.rst #7772

Merged
merged 4 commits into from
Feb 28, 2025
Merged

Update sso-saml-technical.rst #7772

merged 4 commits into from
Feb 28, 2025

Conversation

jlandells
Copy link
Contributor

Added details of the SAML Chrome Panel extension for troubleshooting.

Summary

Ticket Link

Added details of the SAML Chrome Panel extension for troubleshooting.
@jlandells jlandells requested a review from cwarnermm February 28, 2025 14:56
Copy link

Newest code from mattermost has been published to preview environment for Git SHA 5b7dde4

@jlandells
Copy link
Contributor Author

I just spotted that the link is not formed correctly for RST. I'll go back and fix it.

Changed link to be the same format as other links (not Markdown!)
Copy link

Newest code from mattermost has been published to preview environment for Git SHA 17dfa4d

Trying to encourage the link to be a link
It needs a trailing underscore??? 🤯
Copy link

Newest code from mattermost has been published to preview environment for Git SHA 26b3d43

Copy link

Newest code from mattermost has been published to preview environment for Git SHA d117f23

Copy link
Member

@cwarnermm cwarnermm left a comment

Choose a reason for hiding this comment

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

Thanks, @jlandells!

@cwarnermm cwarnermm added the 3: Reviews Complete All reviewers have approved the pull request label Feb 28, 2025
@cwarnermm cwarnermm merged commit d04d7ea into master Feb 28, 2025
4 checks passed
@cwarnermm cwarnermm deleted the jlandells-patch-1 branch February 28, 2025 15:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3: Reviews Complete All reviewers have approved the pull request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants