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

Users receive 404 error on For Developers and Figma Guide Pages on Documentation site #1412

Closed
bennettcc opened this issue Nov 30, 2023 · 0 comments · Fixed by #1413
Closed
Assignees
Labels
Needs Technical Review Needs review from Architect and Development

Comments

@bennettcc
Copy link

bennettcc commented Nov 30, 2023

Anna has reported issues on the documentation site - she is receiving 404 errors when viewing the following two pages:

Developers: https://designsystem-dev.cancer.gov/develop/developers
Figma Guide: https://designsystem-dev.cancer.gov/develop/get-started/figma-guide

The problem seems to be from the side nav (first parent top level link) that removes the slash and causes a pathing issue. or if you go to the URL directly from a link without the slash.

The sidenav needs to be altered to stop removing the slash from this top level link.

The behavior seems to be inconsistent - upon testing I'm not receiving the 404 error every time I access the page. Testing these scenarios:

  • Navigating directly to the URL via the address bar
  • Navigate to the content pages via the left nav under the 'Get Started' section
  • Navigate to the content pages by clicking the H2 links on the 'Get Started' landing page

image

@karlikpj karlikpj self-assigned this Nov 30, 2023
karlikpj added a commit that referenced this issue Nov 30, 2023
karlikpj added a commit that referenced this issue Nov 30, 2023
karlikpj added a commit that referenced this issue Nov 30, 2023
karlikpj added a commit that referenced this issue Dec 1, 2023
@bennettcc bennettcc added the Needs Technical Review Needs review from Architect and Development label Dec 1, 2023
karlikpj added a commit that referenced this issue Dec 1, 2023
karlikpj added a commit that referenced this issue Dec 1, 2023
karlikpj added a commit that referenced this issue Dec 1, 2023
arcepaul pushed a commit that referenced this issue Dec 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needs Technical Review Needs review from Architect and Development
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants