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

[storybook]: broken usage guidelines links #5837

Closed
9 tasks
davidmenendez opened this issue Aug 8, 2024 · 6 comments
Closed
9 tasks

[storybook]: broken usage guidelines links #5837

davidmenendez opened this issue Aug 8, 2024 · 6 comments
Assignees
Labels
type: bug 🐛 Something isn't working type: stories 📚 Updates to Storybook examples or docs

Comments

@davidmenendez
Copy link
Contributor

davidmenendez commented Aug 8, 2024

Package

Carbon for IBM Products

Description

minor thing here i've noticed throughout our storybook. there appears to be numerous broken links throughout the storybook. probably because the website was updated at some point. AboutModal and RemoveModal being just two examples

Component(s) impacted

i haven't audited every single component and pattern, but AboutModal and RemoveModal for starters

Browser

Chrome

@carbon/ibm-products (previously @carbon/ibm-cloud-cognitive) version

2.47

Suggested Severity

Severity 3 = The problem is visible or noticeable to users but does not impede the usability or functionality. Affects minor functionality, has a workaround.

Product/offering

carbon products

CodeSandbox or Stackblitz example

NA

Steps to reproduce the issue (if applicable)

  1. Go to storybook
  2. Go to docs tab of component or pattern
  3. Verify that the required usage guidelines link is going to the appropriate place

Release date (if applicable)

No response

Code of Conduct

Components that have custom guidelines url

Preview Give feedback
@matthewgallo
Copy link
Member

I started compiling a list but after further investigation, I think most of the issues come from how the url is created here.

One issue in particular is that for the "Components" section in our storybook the kind value will be Components, but the way the url is constructed now will cause it to be the following:

https://pages.github.ibm.com/cdai-design/pal/Componentss/${paramCase(...)}

the capital "C" and the extra s in Componentss cause 404s.

@matthewgallo matthewgallo moved this from Needs triage 🧐 to Backlog 🌋 in Carbon for IBM Products Aug 12, 2024
@davidmenendez
Copy link
Contributor Author

@davidmenendez
Copy link
Contributor Author

shout out to @janhassel for implementing a fix that handles some of the broken urls.

i need to point out this comment i made that addresses the fact that this didn't solve everything though. we're still seeing some links that are broken in-spite of the capitalization and extra characters fix

@matthewgallo
Copy link
Member

For those other components where the url doesn't match the standard, we can use the altGuidelinesHref prop in the <StoryDocsPage /> component to specify the exact url it should point to.

@ljcarot
Copy link
Member

ljcarot commented Jan 2, 2025

@davidmenendez or @matthewgallo what is the status of this issue?

@matthewgallo
Copy link
Member

We just need to do an audit because there are some that still result in 404s, but the main issue causing the 404s was fixed in #5855. I think we can close this and open a new issue to audit for any remaining 404 links.

@github-project-automation github-project-automation bot moved this from In review 👀 to Done 🚀 in Carbon for IBM Products Jan 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug 🐛 Something isn't working type: stories 📚 Updates to Storybook examples or docs
Projects
Status: Done 🚀
Development

No branches or pull requests

4 participants