-
Notifications
You must be signed in to change notification settings - Fork 28
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
memory leak: Column API Function getColumnState() cannot be called as the grid has been destroyed #233
Comments
Hello @dragonite00, Could you please provide a working example of how you encounter this issue? |
hey, thanks for the response! The setup is basically, I have a (pseudocode)
if I click between the checklist values too quickly (actually it takes ~2s to return the ag grids so I'm not even clicking that fast), then I get the aforementioned error. along with a |
Hmm, interesting, have you tried using Patch with pre-existing grids? In this style, I think you are encountering the issue because you are removing the grid before it can be fully spun up. We can take a look as see if there is something we can do to remove the events if the grid is removed before completely available. Most api and columnApi functions will only occur after the grid is completely up, unless you are then trying to pass it data or columnDefs immediately... You could also potentially use the new |
hmm, if you mean have a fixed set of grids, it's not possible because the checklist also controls the number of grids
this would be amazing, since this pattern seems most "intuitive" to me
Hm, I thought passing columnDefs in the |
Can you please provide a working example of this? I cannot dig into what is happening unless there is something to test with. Are you also passing all the data and stuff to the grids too? |
Hello @dragonite00, Could you give this a test on the v31 of the grid to see if this issue persists? |
I have a checklist that updates the page with a different set of ag-grid depending on the selected option. I get the following error
The text was updated successfully, but these errors were encountered: