-
Notifications
You must be signed in to change notification settings - Fork 406
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
Resolve broken links in new docs site #1153
Comments
@bmuenzenmeyer A comment and question:
Do these need to have the same URL structure as the live site or is it okay for these to be different? I think most of the URLs posted above from the new site are just because the title has been renamed. |
Good questions. The crawler found those links somewhere on the new site and they broke. Navigate around and you will find some. The goal is to ferret these out wherever they are |
@bmuenzenmeyer Alrighty. I found the issue with resources. I'll take a look at the others and fix them if they need to be fixed. |
@bmuenzenmeyer @bradfrost For the documentation page, do we still want that since all of our docs are in the nav now? |
@bmuenzenmeyer @bradfrost In terms of the new site:
Assuming those are the ones we should focus on since we are launching the new site soon. |
@bmuenzenmeyer Just created a pull request for this. All of the new links should be resolved. The old links from above have either been renamed or no longer have any significance. |
I used https://validator.w3.org/checklink to checkout our new doc site, previewed at: https://patternlab-docs-preview.netlify.app/
This is prerequisite for cutting over the docs site.
There are a number of broken links we need to resolve. My best attempt at scraping the w3c results and deduping looks like this:
https://patternlab-docs-preview.netlify.app/resources
https://patternlab.io//docs/template-language-and-patternengines/
https://patternlab-docs-preview.netlify.app/resources/
https://patternlab.io//docs/starterkits/
https://patternlab-docs-preview.netlify.app/docs/pattern-reorganizing.html
https://patternlab-docs-preview.netlify.app/docs
https://patternlab.io//feed.xml
https://patternlab.io//docs/using-pattern-states/
https://patternlab.io//updates/
https://patternlab.io//docs/managing-pattern-assets/
https://patternlab.io//docs/creating-pattern-specific-values/
https://patternlab.io//docs/modifying-the-pattern-header-and-footer/
https://patternlab-docs-preview.netlify.app/docs/advanced-auto-regenerate.html
https://patternlab-docs-preview.netlify.app/docs/pattern-including.html
https://patternlab.io//docs/keyboard-shortcuts/
https://patternlab.io//docs/using-pseudo-patterns/
cc @frostyweather @bradfrost
The text was updated successfully, but these errors were encountered: