-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Handle removal from the DOM when integrated in React apps #8090
Comments
@willemarcel Thanks for opening this, it's a big bug! I wouldn't be surprised if iD isn't caching/uncaching things properly since it hasn't historically been restarted like this, but we'll need to look further into it. |
not sure if it helps, but i've got a related report from a user (@tastrax) of the oceania tasking manager. he gave me this stack trace. i checked the network traffic and it looks ok (i.e. nodes before ways from OSM's API).
|
@jayenashar I'm solving it with hotosm/tasking-manager#3792 |
fantastic. i'll pull it into our TM so @tastrax can use it |
🕺 |
We are having some problems when running iD inside Tasking Manager.
What I noticed is that if I have a task open on iD in TM, then I move to another React view and go back (both using the back button or selecting to resume the task), iD will load but won't download the data again. It's working like if we didn't restart iD, but just went back to the instance that were running (although the OSM data is not rendered). Users reported also that if they try to load a neighbour task, iD doesn't download data.
Maybe a solution is to add a destroy action on iD that would remove it from the DOM, like it's explained on: https://reactjs.org/docs/integrating-with-other-libraries.html#integrating-with-dom-manipulation-plugins
The text was updated successfully, but these errors were encountered: