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] Re-enable storysource addon #10363

Merged
merged 6 commits into from
Nov 27, 2024
Merged

[Storybook] Re-enable storysource addon #10363

merged 6 commits into from
Nov 27, 2024

Conversation

slax57
Copy link
Contributor

@slax57 slax57 commented Nov 20, 2024

Problem

Storybook storysource addon (allowing to display the source code for a story in the 'Code' panel) was broken since we updated to Storybook v8.

It would be nice to re-enable it to allow viewers of our storybook to also view the matching source code.

Solution

I actually managed to track down a bug in Storybook, preventing this addon to work in the first place.

This PR re-enables the plugin, and is to be merged when the issue is fixed in a new storybook release.

Blocked: wait until storybookjs/storybook#29669 is merged and released
=> EDIT: No news from storybook maintainers, so this PR uses a local patch instead.

How To Test

Run storybook, and check the source code of each story is present in the 'Code' panel.

https://react-admin-storybook-i85occt9g-marmelab.vercel.app/?path=/story/ra-core-form-filterliveform--basic

@slax57 slax57 added WIP Work In Progress DNM Do Not Merge labels Nov 20, 2024
@slax57 slax57 changed the title [DNM] [Storybook] Re-enable storysource addon [Storybook] Re-enable storysource addon Nov 27, 2024
@slax57 slax57 removed WIP Work In Progress DNM Do Not Merge labels Nov 27, 2024
@slax57 slax57 added the RFR Ready For Review label Nov 27, 2024
@erwanMarmelab erwanMarmelab self-requested a review November 27, 2024 13:39
@fzaninotto fzaninotto merged commit bc4a1e0 into master Nov 27, 2024
15 checks passed
@fzaninotto fzaninotto deleted the addon-storysource branch November 27, 2024 16:43
@fzaninotto fzaninotto added this to the 5.4.1 milestone Nov 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
RFR Ready For Review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants