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: infinite render and console errors due to search component #503

Conversation

Spiral-Memory
Copy link
Collaborator

@Spiral-Memory Spiral-Memory commented Mar 8, 2024

Brief Title

I have observed some minor bugs introduced by PR #497 , and this PR is intended to address these issues of infinite renders and console errors as quickly as possible

Acceptance Criteria Fulfillment

  • Resolved infinite renders
  • Resolved console error related to SVG size

Fixes #502

Video/Screenshots

2024-03-08.22-16-36.mp4

@Spiral-Memory
Copy link
Collaborator Author

Hey @sidmohanty11 , there is a minor bug introduced after recent merge #497 while rendering the search component. It was rendering infinitely. I have tried fixing it. Please review this as early as possible.

@JeffreytheCoder
Copy link
Contributor

@Spiral-Memory Thanks for spotting this 👍

@Spiral-Memory
Copy link
Collaborator Author

@Spiral-Memory Thanks for spotting this 👍

No worries, bro 😃! We are here to help each other.

Copy link
Collaborator

@sidmohanty11 sidmohanty11 left a comment

Choose a reason for hiding this comment

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

Thanks a lot @Spiral-Memory good catch!

@sidmohanty11 sidmohanty11 merged commit 9997a90 into RocketChat:develop Mar 8, 2024
3 checks passed
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.

[BUG] Infinite render of search message component
3 participants