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

Research best practices around Hoist components used as grid column renderers #3311

Open
PeteDarinzo opened this issue Mar 16, 2023 · 0 comments

Comments

@PeteDarinzo
Copy link
Contributor

When a Hoist component with a local model is used as an AG Grid column renderer, it is remounted on every prop update.

This issue was discovered when using a component that displays the elapsed time between the current time and a timestamp prop, similar to Hoist's relativeTimestamp component.

Remounting of the component every time the a new timestamp was received caused a memory leak which was visible from the Windows task manager processes tab. The issue was corrected by making one timer applicable to the tab model that contained the grid, but the issue could be reappear in other applications.

@amcclain amcclain changed the title Hoist components used as column renderers receive multiple remounts upon prop change. Research best practices around Hoist components used as grid column renderers Jun 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants