Skip to content

Conversation

@vdice
Copy link
Member

@vdice vdice commented Aug 5, 2025

  • For some reason, the relative /deploy link was considered broken by the link checker. Perhaps because there are multiple deploy.md files? In any case, updating to point to the version under wasm-functions appears to fix it...

@github-actions
Copy link

github-actions bot commented Aug 5, 2025

🚀 preview deployed successfully to Fermyon Cloud and available at https://fermyon-developer-pr-1564-deozwrmi.fermyon.app

@karthik2804
Copy link
Contributor

/deploy translates is an absolute link from the root of the domain. So it translates to localhost:3000/deploy which would look for a file located at content/deploy.md which does not exist. The relative link needs to be ./deploy.

@vdice
Copy link
Member Author

vdice commented Aug 5, 2025

Hmmm, I tried updating to both ./deploy#... and deploy#... and these both resulted in the same error 🤔

@karthik2804
Copy link
Contributor

That might indicate some error in the absolutization logic in bartholomew then. I can take a peek. But for now, we have been defaulting to using absolute urls anyway because of some other issue with how browsers resolved links when the page url ended with a /.

@vdice vdice merged commit 40a08c5 into main Aug 5, 2025
3 checks passed
@vdice vdice deleted the fix/broken-link branch August 5, 2025 09:45
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.

2 participants