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

Error 404 for 14 pages after deploy of restructure #186

Closed
mcking65 opened this issue Feb 1, 2023 · 3 comments
Closed

Error 404 for 14 pages after deploy of restructure #186

mcking65 opened this issue Feb 1, 2023 · 3 comments
Assignees
Labels
bug Something isn't working Infrastructure

Comments

@alflennik
Copy link
Contributor

@mcking65 @howard-e I opened a PR #187 which fixes this issue. Once it's reviewed and merged we can request a follow-on deploy from the WAI team.

Regarding two of the broken links, the datagrids link and the layoutgrids links, I didn't make any changes to those pages because the links as written above have incorrect capitalization, they should be dataGrids and layoutGrids instead of datagrids and layoutgrids. The pages within the site do have the correct capitalization so I don't think those two pages are actually broken.

Finally, there is another issue which my PR doesn't address and we still have to fix: there are now broken image links in the feedDisplay page.

That example dynamically creates images, which, as of the release yesterday, is not something we support, because on the final deploy the images will be stored in a different directory than the source code. All of the content and assets for the entire site will be split up into three groups, a content folder for html files, a content-images folder for images, and a content-assets folder for everything else.

Personally, while the site does currently work, I think the mental overhead of making sure the APG supports its html, images and assets being automatically divided up into three directories when deployed is not a good long-term solution.

@alflennik
Copy link
Contributor

I looked into the datagrids and layout grids a little bit more and found that the WAI website seems to automatically remove capitalized letters from URLs. This is why the links on the site work but bookmarked pages don't.

I think the pages already have a bit of a strange naming convention, given that no other pages have capitalized letters, so a good solution would probably be to rename those two files in the aria-practices repo and introduce new redirects from the old urls to the new ones.

@mcking65
Copy link
Contributor Author

Since the links within the site are now working, I'm closing this issue. I have opened #192 to address the 4 remaining problems related to bookmarks that yield 404s.

mcking65 pushed a commit to w3c/aria-practices that referenced this issue Feb 22, 2023
…ing (pull #2597)

Enable resolution of w3c/wai-aria-practices#186 with the following changes:
* Rename HTML files of grid examples (e.g. dataGrids.html > data-grids.html)
* Rename feedDisplay.html to feed-display.html
* Update hrefs of links to changed file names
* Update tests to support example file name changes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Infrastructure
Projects
None yet
Development

No branches or pull requests

3 participants