Skip to content

Fixed broken link #6522

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

Closed
wants to merge 1 commit into from
Closed

Fixed broken link #6522

wants to merge 1 commit into from

Conversation

rhamedy
Copy link
Contributor

@rhamedy rhamedy commented Feb 11, 2019

Fixes: #6521

Summary: Updated the Pull Request link in the README.adoc to point to the correct Github article https://help.github.com/articles/creating-a-pull-request instead of https://help.github.com/send-pull-requests which is broken.

This link https://docs.spring.io/spring-security/site/docs/current/apidocs/ under https://github.com/spring-projects/spring-security/blob/master/README.adoc#documentation as Spring Security API Documentation is also broken. Added the correct URL https://docs.spring.io/spring-security/site/docs/current/api/

Also noticed that the initial PR had nickname instead of full name. I also squashed the commits to 1.

@rwinch rwinch self-assigned this Feb 15, 2019
@rwinch rwinch added in: docs An issue in Documentation or samples type: bug A general bug labels Feb 15, 2019
@rwinch rwinch added this to the 5.2.0.M2 milestone Feb 15, 2019
@rwinch rwinch added the status: duplicate A duplicate of another issue label Feb 15, 2019
@rwinch
Copy link
Member

rwinch commented Feb 15, 2019

Thanks for the PR @rhamedy! This is merged via 43587b4

PS: I changed the commit message to include that it fixes the issue you logged to follow our conventions. See the commit message to align with the Spring Security conventions?

@rwinch rwinch closed this Feb 15, 2019
@rhamedy rhamedy deleted the gh-6521 branch June 21, 2020 18:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in: docs An issue in Documentation or samples status: duplicate A duplicate of another issue type: bug A general bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants