Proposal: Standardizing Storybook Coverage for Plugin Components #4656
Replies: 2 comments
|
Thanks for the contribution, I will look into the pattern you mentioned! I also agree here, having new (and old) components updated to follow standardized Storybook stories guidelines would be beneficial. I will add this to our next team discussion 👍 |
|
Thanks for the feedback — I’m really glad the idea makes sense in the context of standardizing Storybook stories. |
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
While contributing Storybook coverage for the AI-Assistant plugin, I noticed that several core plugin components lack visual documentation for edge cases like Loading, Error, and Empty states.
The Proposal: > I suggest we make Storybook stories mandatory for all new plugin UI components. This ensures:
UI Consistency: Developers can verify components across different themes and screen sizes without a running cluster.
Robustness: Mandatory 'Error' and 'Loading' stories force us to handle these states gracefully in the code.
I have already implemented this pattern in PR #4655 using the TestContext decorator to provide proper Redux/Router context. I’m happy to help document this standard for other contributors
All reactions