Skip to content

Conversation

@matmilbury
Copy link
Contributor

@matmilbury matmilbury commented Aug 6, 2025

Closes #

What I did

Checklist for Contributors

Testing

The changes in this PR are covered in the following automated tests:

  • stories
  • unit tests
  • integration tests
  • end-to-end tests

Manual testing

This section is mandatory for all contributions. If you believe no manual test is necessary, please state so explicitly. Thanks!

Documentation

  • Add or update documentation reflecting your changes
  • If you are deprecating/removing a feature, make sure to update
    MIGRATION.MD

Checklist for Maintainers

  • When this PR is ready for testing, make sure to add ci:normal, ci:merged or ci:daily GH label to it to run a specific set of sandboxes. The particular set of sandboxes can be found in code/lib/cli-storybook/src/sandbox-templates.ts

  • Make sure this PR contains one of the labels below:

    Available labels
    • bug: Internal changes that fixes incorrect behavior.
    • maintenance: User-facing maintenance tasks.
    • dependencies: Upgrading (sometimes downgrading) dependencies.
    • build: Internal-facing build tooling & test updates. Will not show up in release changelog.
    • cleanup: Minor cleanup style change. Will not show up in release changelog.
    • documentation: Documentation only changes. Will not show up in release changelog.
    • feature request: Introducing a new feature.
    • BREAKING CHANGE: Changes that break compatibility in some way with current major version.
    • other: Changes that don't fit in the above categories.

🦋 Canary release

This PR does not have a canary release associated. You can request a canary release of this pull request by mentioning the @storybookjs/core team here.

core team members can create a canary release here or locally with gh workflow run --repo storybookjs/storybook canary-release-pr.yml --field pr=<PR_NUMBER>

Greptile Summary

This PR fixes a broken documentation link in the CHANGELOG.md file. The change updates a single URL reference from the old path format /docs/9/migration-guide to the new path format /docs/releases/migration-guide. This appears to be part of a documentation restructuring where Storybook moved their migration guide from version-specific paths to a more generic releases section.

The CHANGELOG.md file serves as a comprehensive record of all changes across Storybook versions and contains important links that users rely on when upgrading between versions. This particular link directs users to the migration guide, which is critical for understanding breaking changes and upgrade procedures. The fix ensures that users clicking on this link will reach the correct documentation page rather than encountering a 404 error.

PR Description Notes:

  • The PR description is incomplete, with empty sections for "What I did" and "Closes #" fields
  • None of the testing checkboxes are marked, though manual testing may not be necessary for a simple link fix
  • The documentation checkbox is not marked despite this being a documentation change

Confidence score: 5/5

  • This PR is extremely safe to merge with virtually no risk of causing issues
  • Score reflects the minimal nature of the change - a single URL correction that cannot break functionality
  • No files require special attention as this is a straightforward documentation link fix

Context used:

Context - In the documentation, ensure that the links to the builder's documentation and the Next.js framework documentation are correct and not broken. (link)

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

1 file reviewed, no comments

Edit Code Review Bot Settings | Greptile

@jonniebigodes jonniebigodes self-assigned this Aug 6, 2025
@jonniebigodes jonniebigodes added documentation ci:docs Run the CI jobs for documentation checks only. labels Aug 6, 2025
@nx-cloud
Copy link

nx-cloud bot commented Aug 6, 2025

View your CI Pipeline Execution ↗ for commit c1e5cf5

Command Status Duration Result
nx run-many -t build --parallel=3 ✅ Succeeded 1m 7s View ↗

☁️ Nx Cloud last updated this comment at 2025-08-06 19:37:53 UTC

@yannbf yannbf changed the title Update CHANGELOG.md - fix a dead link Documentation: fix migration guide link in changelog Aug 8, 2025
@yannbf yannbf enabled auto-merge August 8, 2025 13:05
@yannbf
Copy link
Member

yannbf commented Aug 8, 2025

Thanks a lot for your contribution!

@yannbf yannbf disabled auto-merge August 8, 2025 13:06
@yannbf yannbf merged commit e0d33e3 into storybookjs:next Aug 8, 2025
8 of 12 checks passed
@github-actions github-actions bot mentioned this pull request Aug 8, 2025
14 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci:docs Run the CI jobs for documentation checks only. documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants