[openstreetmap/openstreetmap-website] Note subscriptions API (PR #5314)

Anton Khorev notifications at github.com
Wed Nov 20 16:38:28 UTC 2024


> PR contains a fixup commit

Let me guess what's a *fixup* here. The fact that looking up the note is not required in `destroy` if there aren't two different error messages (*note not found* or *not subscribed*)? Now I changed it to always look up the note. Is there still a fixup?

-- 
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/5314#issuecomment-2489070243
You are receiving this because you are subscribed to this thread.

Message ID: <openstreetmap/openstreetmap-website/pull/5314/c2489070243 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20241120/5e170a30/attachment.htm>


More information about the rails-dev mailing list