Skip to content

Fix invalid link in transaction resources #30570

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

Conversation

kihyuk-sung
Copy link
Contributor

@kihyuk-sung kihyuk-sung commented May 31, 2023

Hello. I found that Distributed transactions in Spring, with and without XA url in Spring Framework - Data Access - Transaction Management - Further Resources is not linked to the valid url.
https://docs.spring.io/spring-framework/reference/data-access/transaction/resources.html

https://www.infoworld.com/article/2077963/distributed-transactions-in-spring--with-and-without-xa.html

This url contains double dash (--). In that case, asciidoctor use some specific methods described in followed links.

https://docs.asciidoctor.org/asciidoc/latest/macros/complex-urls/
asciidoctor/asciidoctor#3965

I fix this problem using link prefix and double plus line macro(++) because it preserve the original url.

There are another solutions to solve this problem. If you don't like this way, please edit this document page.

@pivotal-cla
Copy link

@kihyuk-sung Please sign the Contributor License Agreement!

Click here to manually synchronize the status of this Pull Request.

See the FAQ for frequently asked questions.

@pivotal-cla
Copy link

@kihyuk-sung Thank you for signing the Contributor License Agreement!

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged or decided on label May 31, 2023
@snicoll snicoll added type: documentation A documentation task and removed status: waiting-for-triage An issue we've not yet triaged or decided on labels May 31, 2023
@snicoll snicoll added this to the 6.0.x milestone May 31, 2023
@snicoll snicoll self-assigned this May 31, 2023
@snicoll snicoll modified the milestones: 6.0.x, 6.0.10 May 31, 2023
snicoll pushed a commit that referenced this pull request May 31, 2023
snicoll added a commit that referenced this pull request May 31, 2023
@snicoll snicoll closed this in 5b471a5 May 31, 2023
@snicoll
Copy link
Member

snicoll commented May 31, 2023

Good catch, thanks @kihyuk-sung!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: documentation A documentation task
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants