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

[Bug?]: Storybook preview config load not working #10113

Closed
1 task done
ChrisLFieldsII opened this issue Mar 5, 2024 · 4 comments · Fixed by #10160
Closed
1 task done

[Bug?]: Storybook preview config load not working #10113

ChrisLFieldsII opened this issue Mar 5, 2024 · 4 comments · Fixed by #10160
Labels
bug/needs-info More information is needed for reproduction

Comments

@ChrisLFieldsII
Copy link

ChrisLFieldsII commented Mar 5, 2024

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

Screenshot 2024-03-05 at 2 41 05 AM

How do we reproduce the bug?

Create a bare redwood project with the provided rw info and add logs to storybook.preview file and notice the logs never show in the web dev console as they should

I personally added logs to the @redwoodjs/testing node_module here and noticed that the if block was never hit.

Attached are before/after pics once I patched the if condition

Before
before

After
after

What's your environment? (If it applies)

System:
    OS: macOS 14.2.1
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 20.11.1 - /private/var/folders/4h/7lryc_8j0dnddhpng25wx3dw0000gn/T/xfs-063fe80d/node
    Yarn: 4.1.0 - /private/var/folders/4h/7lryc_8j0dnddhpng25wx3dw0000gn/T/xfs-063fe80d/yarn
  Databases:
    SQLite: 3.43.2 - /usr/bin/sqlite3
  Browsers:
    Chrome: 122.0.6261.94
    Safari: 17.2.1
  npmPackages:
    @redwoodjs/cli-storybook: 7.0.4 => 7.0.4 
    @redwoodjs/core: 7.0.4 => 7.0.4 
    @redwoodjs/project-config: 7.0.4 => 7.0.4

Are you interested in working on this?

  • I'm interested in working on this
@ChrisLFieldsII ChrisLFieldsII added the bug/needs-info More information is needed for reproduction label Mar 5, 2024
ChrisLFieldsII referenced this issue Mar 5, 2024
Co-authored-by: Daniel Choudhury <dannychoudhury@gmail.com>
@ahaywood
Copy link
Contributor

ahaywood commented Mar 5, 2024

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.

@arimendelow
Copy link
Contributor

Hi @ChrisLFieldsII! I'm taking a look. Will keep you posted :)

@ChrisLFieldsII ChrisLFieldsII changed the title [Bug?]: Storybook preview config load broke [Bug?]: Storybook preview config load not working Mar 6, 2024
Josh-Walker-GM pushed a commit that referenced this issue Mar 9, 2024
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.
jtoar pushed a commit that referenced this issue Mar 9, 2024
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.
jtoar pushed a commit that referenced this issue Mar 9, 2024
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.
@arimendelow
Copy link
Contributor

Hi @ChrisLFieldsII! Just looping back here to check that you saw this was fixed.

@ChrisLFieldsII
Copy link
Author

Yes! Amazing work. Much appreciated

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug/needs-info More information is needed for reproduction
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants