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

broken links #51

Closed
rajatjindal opened this issue Oct 12, 2022 · 6 comments
Closed

broken links #51

rajatjindal opened this issue Oct 12, 2022 · 6 comments

Comments

@rajatjindal
Copy link
Contributor

in spin/* pages, the link to other pages under spin is currently broken. e.g.

noticed that its broken similarly in other pages as well.

@rajatjindal
Copy link
Contributor Author

it might be related to how bartholomew handles relative links.

@rajatjindal
Copy link
Contributor Author

opened PR: fermyon/bartholomew#136 which hopefully fixes this issue

@rajatjindal
Copy link
Contributor Author

after the fix:

blc --recursive http://127.0.0.1:3000 --exclude-external 
Getting links from: http://127.0.0.1:3000/
├───OK─── http://127.0.0.1:3000/spin
├───OK─── http://127.0.0.1:3000/cloud
Finished! 24 links found. 22 excluded. 0 broken.

Getting links from: http://127.0.0.1:3000/spin
└───OK─── http://127.0.0.1:3000/spin/quickstart
Finished! 42 links found. 41 excluded. 0 broken.

Getting links from: http://127.0.0.1:3000/cloud
Finished! 47 links found. 47 excluded. 0 broken.

Getting links from: http://127.0.0.1:3000/spin/quickstart
├───OK─── http://127.0.0.1:3000/spin/contributing
├───OK─── http://127.0.0.1:3000/spin/configuration
├───OK─── http://127.0.0.1:3000/spin/rust-components
├───OK─── http://127.0.0.1:3000/spin/go-components
├───OK─── http://127.0.0.1:3000/spin/developing
Finished! 56 links found. 51 excluded. 0 broken.

Getting links from: http://127.0.0.1:3000/spin/contributing
└───OK─── http://127.0.0.1:3000/spin/extending-and-embedding
Finished! 53 links found. 52 excluded. 0 broken.

Getting links from: http://127.0.0.1:3000/spin/configuration
├───OK─── http://127.0.0.1:3000/spin/http-trigger
└───OK─── http://127.0.0.1:3000/spin/redis-trigger
Finished! 53 links found. 51 excluded. 0 broken.

Getting links from: http://127.0.0.1:3000/spin/rust-components
Finished! 53 links found. 53 excluded. 0 broken.

Getting links from: http://127.0.0.1:3000/spin/go-components
Finished! 52 links found. 52 excluded. 0 broken.

Getting links from: http://127.0.0.1:3000/spin/developing
Finished! 41 links found. 41 excluded. 0 broken.

Getting links from: http://127.0.0.1:3000/spin/extending-and-embedding
Finished! 54 links found. 54 excluded. 0 broken.

Getting links from: http://127.0.0.1:3000/spin/http-trigger
├───OK─── http://127.0.0.1:3000/spin/distributing-apps
Finished! 57 links found. 56 excluded. 0 broken.

Getting links from: http://127.0.0.1:3000/spin/redis-trigger
Finished! 51 links found. 51 excluded. 0 broken.

Getting links from: http://127.0.0.1:3000/spin/distributing-apps
Finished! 46 links found. 46 excluded. 0 broken.

Finished! 629 links found. 617 excluded. 0 broken.
Elapsed time: 1 second

we should add broken checks as github action as well.

@rajatjindal
Copy link
Contributor Author

we also probably need to fix sidebar nav from:

<li><a href="{{site.info.base_url}}/spin/other-languages/">Other languages</a></li>

to

<li><a href="{{site.info.base_url}}/spin/other-languages">Other languages</a></li>

Notice the / missing after other-languages

@rajatjindal
Copy link
Contributor Author

the bartholomew PR is merged, we should pull in new wasm for it

@karthik2804
Copy link
Contributor

closed with #53

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

No branches or pull requests

2 participants