[openstreetmap/openstreetmap-website] Transition more requests to fetch (PR #5669)
Marwin Hochfelsner
notifications at github.com
Fri Feb 14 05:20:43 UTC 2025
@hlfan commented on this pull request.
>
- noteLoader = null;
+ for (var id in oldNotes) {
+ noteLayer.removeLayer(oldNotes[id]);
+ }
+ })
+ .catch(() => {})
> have a no-op catch on all of them?
Currently, I only put catches on fetches with AbortSignals, cause these are the errors I can catch.
Network errors will still end up in the console.
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/5669#discussion_r1955572213
You are receiving this because you are subscribed to this thread.
Message ID: <openstreetmap/openstreetmap-website/pull/5669/review/2616790205 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20250213/31f79d7e/attachment.htm>
More information about the rails-dev
mailing list