-
Notifications
You must be signed in to change notification settings - Fork 17
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
Invalid hook call error when using addon Knobs "text" knob #17
Comments
FYI, we’ve released Please upgrade and try them out today! |
@shilman Noted! I didn't realise they were intended to replace |
@Sniffing even short term for many! 😉 controls is still missing a couple features from knobs, but it also adds a few big ones and is already better in many ways (simpler, probably more stable, maintained). |
Pulled it down, gave it a try, most of the controls seem good, I still seem to be having the issue above when using
|
I'm getting the same error as the comment above when attempting to use I edited Then with the debugger I can see I haven't used webpack DLLs before. Is that DLL packaging its own copy of React that doesn't show up in my |
I think my problem was that I had several
|
Yeah, we have a problem when third party addons have not upgraded their dependencies across breaking changes (5.3.x => 6.0.x) @ndelangen @tmeasday any thoughts on how we can make this easier for users? |
@shilman we don't have exactly a good place for this. As a short term solution we could use the FAQ page (workflows/faq.md) and add a new section addressing this issue. As this issue is addressed we can then move it to a better place. Let me know if you're ok with it and we'll move from there. |
@jonniebigodes that sounds like a great place. 💯 |
Ran into this today when bumping from 6.0.21 to 6.0.22. I experimented with forking @sambego/storybook-state to update the Storybook versions, which then only showed 6.0.22 when running We've got extensive uses of the text control from Are there expected changes with future versions of Storybook that might make it less sensitive to version mismatches? If so, I might pin our version for a while and wait for those updates. |
@stevethatcodes Storybook uses webpack DLLS for performance and this causes lots of version issues, possibly including this one. We're planning on getting rid of them in 6.1: storybookjs/storybook#12637 |
@shilman Aha, thank you for that info, I think we'll pin to |
Sorry for not addressing this change sooner. I had to shift my attention to other documentation. I'm going to make a pull request addressing this. To give a bit more context when using third party addons with the Storybook 6.0. Regarding your issue @Sniffing this is something that might need a bit more context and information. Sorry for not providing a better response but we're in a bit of a pickle as it's a company code. Once the pull request is created i'll link this issue. Sorry for not providing a better answer for you. Stay safe. |
Chore: update documentation to address issue #12388
Chore: update documentation to address issue #12388
Hi everyone! Seems like there hasn't been much going on in this issue lately. If there are still questions, comments, or bugs, please feel free to continue the discussion. Unfortunately, we don't have time to get to every issue. We are always open to contributions so please send us a pull request if you would like to help. Inactive issues will be closed after 30 days. Thanks! |
Describe the bug
Related to:
When using addon knobs, any use of the
text
knob with causes the above error. Removing knobs of this type while keeping other types do not cause the errorTo Reproduce
"@storybook/addon-knobs": "^6.0.21",
Company source code, so cannot link.
withKnobs
decoratortext
Expected behavior
Story for component loads with knob
Code snippets
System:
The text was updated successfully, but these errors were encountered: