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

ui: Fix the crash with the error Rendered fewer hooks than expected. #4712

Merged
merged 1 commit into from
Jun 4, 2024

Conversation

yomete
Copy link
Contributor

@yomete yomete commented Jun 4, 2024

By moving the memoized icicleGraph component to be above the early return statement.

By moving the memoized `icicleGraph` component to be above the early return statement.
Copy link

alwaysmeticulous bot commented Jun 4, 2024

🤖 Meticulous spotted visual differences in 13 of 352 screens tested: view and approve differences detected.

Last updated for commit b0f717f. This comment will update as new commits are pushed.

Copy link
Contributor

@manojVivek manojVivek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lgtm!

@yomete yomete merged commit e0e4868 into main Jun 4, 2024
38 checks passed
@yomete yomete deleted the fix-fewer-hooks-bug branch June 4, 2024 14:27
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