-
Notifications
You must be signed in to change notification settings - Fork 6.2k
Fix docs #18488
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
base: main
Are you sure you want to change the base?
Fix docs #18488
Conversation
|
Hi, @c-arianna! Thank you for the PR. It appears that you have added other cleanup as well, which is great. Will you please place these in separate commits? In this way, the commit that fixes the linked issue only contains those changes. This simplifies future research as well as simplifying backporting. From what I can tell, you have four separate improvements:
Would you please separate these into separate commits? Only the first one should have "Closes gh-18376" at the end. If you have trouble with this, please let me know and I can help. |
Closes spring-projectsgh-18376 Signed-off-by: arianna <[email protected]>
Signed-off-by: arianna <[email protected]>
Signed-off-by: arianna <[email protected]>
- Fix wrong docs link in Servlet - SAML2 Log In Overview - Fix examples in Servlet - SAML2 Authentication Responses - Fix examples in Servlet - SAML2 Logout Signed-off-by: arianna <[email protected]>
|
Hi @jzheaux, sorry about that and thanks for the feedback! |
Closes gh-18376