-
Notifications
You must be signed in to change notification settings - Fork 970
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
[Bug?]: Storybook preview config load not working #10113
Comments
Co-authored-by: Daniel Choudhury <dannychoudhury@gmail.com>
Hey @ChrisLFieldsII Thanks for taking the time to file an issue and provide the screenshots. Those are a huge help. I'm going to pass your issue on to one of the core team members that specializes in Storybook and let you know what I find out. |
Hi @ChrisLFieldsII! I'm taking a look. Will keep you posted :) |
Fixes #10113. I didn't see a pull request for this, but the fix seems to have already been identified in the linked issue and worked in my local testing.
Fixes #10113. I didn't see a pull request for this, but the fix seems to have already been identified in the linked issue and worked in my local testing.
Fixes #10113. I didn't see a pull request for this, but the fix seems to have already been identified in the linked issue and worked in my local testing.
Hi @ChrisLFieldsII! Just looping back here to check that you saw this was fixed. |
Yes! Amazing work. Much appreciated |
What's not working?
The code responsible for loading the storybook preview config is not working due to not using the correct path on the redwoodProjectPaths object
It looks like a recent change introduced this bug.
I attached a pic of the commit that I believe introduced this bug as well as a link
How do we reproduce the bug?
Create a bare redwood project with the provided
rw info
and add logs tostorybook.preview
file and notice the logs never show in the web dev console as they shouldI personally added logs to the
@redwoodjs/testing
node_module here and noticed that theif
block was never hit.Attached are before/after pics once I patched the
if
conditionBefore
After
What's your environment? (If it applies)
Are you interested in working on this?
The text was updated successfully, but these errors were encountered: