-
-
Notifications
You must be signed in to change notification settings - Fork 149
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
List of broken URLs #43
Comments
Do we want to fix all of these? Or just the v6 and v7 links? |
Just v6 for the initial release. I think fixing the archives would be welcomed but not important. I'm re-running the script to know which pages include these broken URLs. |
breakdown of broken URLs: This URL is a weird one listed in every file generated by esdoc:It's already broken on the existing site - not blocking v6 - broken API referencesThe following URLs are esdocs failing to generate the api reference for those files: It's already broken on the existing site - not blocking
The following are broken URLs in Sequelize v3 that could be updated:Not blocking, archived version
The following are fixable broken links on v2:Not blocking, archived version
Broken links on v7Only blocker is this:
|
In the API reference of v6 when you click the bold API Reference link you get referred to the missing page of https://sequelize.org/api/v6/manual/index.html |
I tried fixing the missing pages in the v6 api reference. The only way to repair them would be to update their source code to manually replace the new syntax with syntax compatible with esdoc |
Guess that's what we have to do then unfortunately. Don't think we want to keep that out of the docs for much longer. |
Updated list; #117 (comment) |
@ephys not sure if you shared your script before, but can you run it again? |
My script is available here: https://github.com/ephys/puppeteer-crawler The command I use to check sequelize is: npm start https://sequelize.org -- --collect-meta=anchors \
--exclude-path='/v{1,2,3,4,5}' \
--exclude-path='/v{1,2,3,4,5}/**/*' \
--exclude-path='/api/**/*' \
--exclude-path='/api' \
--check-externals This causes it to check that all pages linked from anything except the above excluded paths are not 404s Then It's a rough script that always requires some tinkering though Here is the latest run:
|
Just ran my script on the latest preview, here is the complete list of broken internal URLs:
The text was updated successfully, but these errors were encountered: