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 integration-more.md #481

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

xbabyx
Copy link
Contributor

@xbabyx xbabyx commented Jan 17, 2025

@polarker
Copy link
Member

It works well for me. Maybe you need to refresh your page?

@xbabyx
Copy link
Contributor Author

xbabyx commented Jan 17, 2025

I think the link works or breaks depending on the current page's URL because relative links are resolved based on the page's location in the site structure.

If you go here https://docs.alephium.org/integration/integration-more and try clicking the "endpoint" under " Transactions
Simple asset transfer" then it will work well.

If you search "Transaction" then go to Troubleshooting under "More and Troubleshooting" it would show that the link is broken.

OLD:
Simple transfer can be created using this [endpoint](./exchange#create-a-transaction).
NEW:
Simple transfer can be created using this [endpoint](../exchange/#create-a-transaction).

Using Absolute link should be a solution to bypass all path issues:
[endpoint](https://docs.alephium.org/integration/exchange/#create-a-transaction)

I always used to get to that part using search so it always used to break for me :P

@polarker
Copy link
Member

I see. Great catch. I will try it locally to test your change 👍

@xbabyx
Copy link
Contributor Author

xbabyx commented Jan 17, 2025

Ty :) mine was also a relative link but different :P so incase mine also cause similar issues on some pages structure, the best solution might be the absolute links that will work from anywhere you click on it!
[endpoint](https://docs.alephium.org/integration/exchange/#create-a-transaction)

@MaudSimon
Copy link
Member

@polarker can we merge this ?

@polarker
Copy link
Member

polarker commented Mar 4, 2025

Sorry, I forgot to work on this. Will do it this afternoon

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

Successfully merging this pull request may close these issues.

3 participants