Remove dangling event listener to avoid memory leaks

[Arooba-git]

Hello 👋
As part of our project, we are using Facebook's new Memlab tool to detect memory leaks in SPA applications.
While running the tool and analyzing the code of TBT, we saw that it does a very good job of ensuring that all async operations are cancelled when the component unmounts. However, as per Memlab execution results, we found that the removal of the event listener is forgotten in one place and is causing the memory to leak (screenshots below).

[before]


Hence we added the fix by removing the event listeners and you can see the heap size and # of leaks reducing noticeably:

You can test this and other potential leak sources, if you like, by running Memlab with a scenario file covering the maximum # of use cases.

Following is a sample of the scenario file we used (it needs to be a .js file but attaching here in txt form):
test-scenario-tbt.txt

Note that some other reported leaks originate from React's internal objects, hence ignored.

[netlify]

✅ Deploy Preview for the-book-thieves ready!

Name	Link
🔨 Latest commit	f210729
🔍 Latest deploy log	https://app.netlify.com/sites/the-book-thieves/deploys/63cec493c8f00c0008036646
😎 Deploy Preview	https://deploy-preview-43--the-book-thieves.netlify.app/
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site settings.

[Arooba-git]

bump @ashikkl