Skip to content

Solid: Update links and docs for framework/renderer package#31928

Merged
shilman merged 8 commits into
storybookjs:nextfrom
kachurun:SolidJS
Aug 11, 2025
Merged

Solid: Update links and docs for framework/renderer package#31928
shilman merged 8 commits into
storybookjs:nextfrom
kachurun:SolidJS

Conversation

@kachurun
Copy link
Copy Markdown
Contributor

@kachurun kachurun commented Jul 2, 2025

What I did

Updated links to SolidJS framework/renderer package.
See https://discord.com/channels/486522875931656193/1381354242333212693

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

Updates all SolidJS framework references across Storybook documentation and configuration files to use 'storybook-solidjs-vite' instead of 'storybook-solidjs', reflecting the package's move to the solidjs-community organization.

  • Updated import statements in 30+ documentation files to use 'storybook-solidjs-vite' for TypeScript type definitions
  • Modified code/core/src/cli/project_types.ts to align renderer and package name with new convention
  • Updated code/lib/cli-storybook/src/sandbox-templates.ts to use new package name and removed 'inDevelopment' flag
  • Changed repository links in README.md and install.mdx to point to solidjs-community/storybook
  • Updated framework package mappings in code/core/src/common/utils/get-storybook-info.ts

Copy link
Copy Markdown
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.

LGTM

49 files reviewed, no comments
Edit PR Review Bot Settings | Greptile

@valentinpalkovic valentinpalkovic added solid ci:normal documentation ci:daily Run the CI jobs that normally run in the daily job. and removed ci:normal labels Jul 2, 2025
@shilman shilman changed the title chore: Updated links and documentation for SolidJS framework/renderer package Solid: Update links and docs for framework/renderer package Jul 2, 2025
@storybook-app-bot
Copy link
Copy Markdown

storybook-app-bot Bot commented Jul 2, 2025

Package Benchmarks

Commit: 0232855, ran on 11 August 2025 at 15:17:48 UTC

No significant changes detected, all good. 👏

@nx-cloud
Copy link
Copy Markdown

nx-cloud Bot commented Jul 3, 2025

View your CI Pipeline Execution ↗ for commit 0232855

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

☁️ Nx Cloud last updated this comment at 2025-08-11 16:30:40 UTC

Copy link
Copy Markdown
Contributor

@jonniebigodes jonniebigodes left a comment

Choose a reason for hiding this comment

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

@kachurun thanks for taking the time to put together this pull request and include the documentation updates. Appreciate it 🙏

From my end, all is good, but I'll defer to @shilman or @valentinpalkovic for the code changes.

Hope you have a great weekend.

Stay safe

@shilman shilman self-assigned this Jul 6, 2025
@kachurun
Copy link
Copy Markdown
Contributor Author

Any updates guys?

@github-actions github-actions Bot added the Stale label Jul 24, 2025
@kachurun
Copy link
Copy Markdown
Contributor Author

kachurun commented Aug 8, 2025

Guys?

@jonniebigodes
Copy link
Copy Markdown
Contributor

@kachurun, sorry for the "radio silence" on this. If you're okay with it, I'll remind @shilman next Monday during our triage meeting if this is not acted upon earlier. Sounds good?

Hope you have a fantastic weekend.

Stay safe

@kachurun
Copy link
Copy Markdown
Contributor Author

kachurun commented Aug 8, 2025

Sure, thank you!

@shilman shilman added the patch:yes Bugfix & documentation PR that need to be picked to main branch label Aug 11, 2025
@shilman shilman merged commit f763c86 into storybookjs:next Aug 11, 2025
76 of 89 checks passed
@github-actions github-actions Bot mentioned this pull request Aug 11, 2025
13 tasks
@github-actions github-actions Bot mentioned this pull request Aug 11, 2025
14 tasks
yannbf pushed a commit that referenced this pull request Aug 11, 2025
Solid: Update links and docs for framework/renderer package
(cherry picked from commit f763c86)
@github-actions github-actions Bot added the patch:done Patch/release PRs already cherry-picked to main/release branch label Aug 11, 2025
@kachurun kachurun deleted the SolidJS branch August 11, 2025 20:04
@ndelangen ndelangen removed the patch:yes Bugfix & documentation PR that need to be picked to main branch label Oct 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci:daily Run the CI jobs that normally run in the daily job. documentation patch:done Patch/release PRs already cherry-picked to main/release branch solid

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants