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

fix(app-webdir-ui): fix rendered more hooks than during previous render #1177

Merged

Conversation

mlsamuelson
Copy link
Member

@davidornelas11 This addresses an error I was seeing in Storybook where if you selected the Search Page story and it rendered, then you select a different story, and then try to go back to the Search Page, you get the error "Rendered more hooks than during previous render".

A search on that error turned up this issue, and the fix in this comment resolved the issue.

This PR implements that fix. Can you verify?

@mlsamuelson
Copy link
Member Author

Turns out, that's happening before the SCHWEB-1089 PR, but we can fix it here.

@scott-williams-az
Copy link
Contributor

Awesome! I noticed that error every now and then, but was not able to track it down.

LGTM

@davidornelas11 davidornelas11 merged commit c367313 into SCHWEB-1089 Oct 9, 2023
@davidornelas11 davidornelas11 deleted the SCHWEB-1089-fix-rendered-more-hooks-error branch October 9, 2023 18:28
@mlsamuelson mlsamuelson restored the SCHWEB-1089-fix-rendered-more-hooks-error branch October 17, 2023 18:40
@mlsamuelson mlsamuelson deleted the SCHWEB-1089-fix-rendered-more-hooks-error branch October 17, 2023 18:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants