chore(site): upgrade storybook to v8 #12831
Conversation
| docs: { | ||
| autodocs: false, | ||
| }, |
There was a problem hiding this comment.
I get why we're starting this off as false, but this looks like it could be a cool feature to flip on at some point
There was a problem hiding this comment.
Maybe?! I see this as helpful for component libraries but not for internal component usage.
| withRouter, | ||
| withQuery, | ||
| (Story) => { | ||
| return ( | ||
| <HelmetProvider> | ||
| <Story /> | ||
| </HelmetProvider> | ||
| ); | ||
| }, |
There was a problem hiding this comment.
Does changing the decorators affect anything here? I would assume this wouldn't break anything, but I'm not sure if there's anything else I'm missing
There was a problem hiding this comment.
Idk why, to be honest, but I had to make the theme provider the last decorator to make it work.
| actions: { | ||
| argTypesRegex: "^(on|handler)[A-Z].*", | ||
| }, |
There was a problem hiding this comment.
I was reading up on this, and I think I get how it works
Did this basically make it so that for any stories that had props that started with on or handle automatically received a Storybook action function as the input?
There was a problem hiding this comment.
I guess so but it is not necessary anymore.
Closes #12682