zeeguu / web

Frontend for the zeeguu web application.
https://www.zeeguu.org
3 stars 5 forks source link

Fixed error of articleListRef.current #411

Closed tfnribeiro closed 1 week ago

tfnribeiro commented 1 week ago

Since this method was being added in an async call, we would sometimes create the event after we would leave the home page. Which leads to it never being cleared.

This implementation always creates the event, and deletes it whenever it's left from the method. The event itself has a guard in case the user attempts to scroll when no more articles are available which avoids making unnecessary calls to the API.

This error doesn't crash or cause problems directly in the app, but it shows up during development, so it would be good to eliminate it!

netlify[bot] commented 1 week ago

Deploy Preview for voluble-nougat-015dd1 ready!

Name Link
Latest commit 6fa2031cf7df282155dbb97a34a03a6ec7483e77
Latest deploy log https://app.netlify.com/sites/voluble-nougat-015dd1/deploys/667e6bd611bc1d0008a286da
Deploy Preview https://deploy-preview-411--voluble-nougat-015dd1.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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