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

chore: Fix links to Reactist components in storybook stories #794

Merged
merged 2 commits into from
Jul 31, 2023

Conversation

engfragui
Copy link
Contributor

@engfragui engfragui commented Jul 31, 2023

Short description

While browsing a couple of Reactist storybook stories, I've noticed that there are some broken links that don't include the path "reactist":

CleanShot.2023-07-31.at.15.17.17.mp4

This PR should fix those links so that do include the correct path and therefore lead the viewer to the right place. (I cannot actually test what will actually happen when this gets deployed to https://doist.github.io/reactist/, as the storybook dev version deployed via Chromatic (linked in the builds below) does not actually include the custom "/reactist/" path in the URL 🤷‍♀️ Either way, this won't break things more than they are now.

PR Checklist

  • Added tests for bugs / new features
  • Updated docs (storybooks, readme)
  • Executed npm run validate and made sure no errors / warnings were shown
  • Described changes in CHANGELOG.md
  • Bumped version in package.json and package-lock.json (npm --no-git-tag-version version <major|minor|patch>) ref
  • Reviewed and approved Chromatic visual regression tests in CI

Versioning

Bumped patch. This is just a doc change.

@engfragui engfragui self-assigned this Jul 31, 2023
@engfragui engfragui added the Show label Jul 31, 2023
@engfragui engfragui requested a review from gnapse July 31, 2023 13:53
@engfragui engfragui merged commit 0bff273 into main Jul 31, 2023
2 checks passed
@engfragui engfragui deleted the francesca/fix-links branch July 31, 2023 13:58
@engfragui
Copy link
Contributor Author

engfragui commented Jul 31, 2023

Whoops, it seems like my fix doesn't actually do anything when testing in Reactist Storybook https://doist.github.io/reactist/?path=/docs/components-input--input-story

I was apparently meant to concat window.location.origin with window.location.pathname 😅

Screenshot 2023-07-31 at 16 15 49

although I'm not sure it will actually do what I want it to do, consistently.

Either way, I will investigate and follow-up on this in #795

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant