[openstreetmap/openstreetmap-website] Bump leaflet.locatecontrol from 0.83.0 to 0.83.1 (PR #5480)

2025-01-07 Thread dependabot[bot] via rails-dev
Bumps [leaflet.locatecontrol](https://github.com/domoritz/leaflet-locatecontrol) from 0.83.0 to 0.83.1.
Commits

Re: [openstreetmap/openstreetmap-website] display warning in new note form after too many anonymous notes (PR #5468)

2025-01-07 Thread etiennejourdier via rails-dev
Thanks for your time, comments, and advice. I just pushed a new version with : - the counter stored in a cookie, with a duration of 30 days, as anonymous visitors don't necessarily post notes every day (I'm not sure about other cookie options, please review) - the cookie read in the controller -

Re: [openstreetmap/openstreetmap-website] display warning in new note form after too many anonymous notes (PR #5468)

2025-01-07 Thread etiennejourdier via rails-dev
@etiennejourdier pushed 2 commits. b12bb162f642a26c03d8743113c27ae775de76e4 display warning in new note form after too many anonymous notes 6e6000c099cdac32f75a03290242ed718b980c3a store anonymous notes counter in a cookie -- View it on GitHub: https://github.com/openstreetmap/openstreetmap-

Re: [openstreetmap/openstreetmap-website] Report deactivates_at as block read date (PR #5478)

2025-01-07 Thread Tom Hughes via rails-dev
Merged #5478 into master. -- Reply to this email directly or view it on GitHub: https://github.com/openstreetmap/openstreetmap-website/pull/5478#event-15843185351 You are receiving this because you are subscribed to this thread. Message ID: ___ rails-

Re: [openstreetmap/openstreetmap-website] Add ElementsController base class (PR #5479)

2025-01-07 Thread Tom Hughes via rails-dev
Merged #5479 into master. -- Reply to this email directly or view it on GitHub: https://github.com/openstreetmap/openstreetmap-website/pull/5479#event-15843185371 You are receiving this because you are subscribed to this thread. Message ID: ___ rails-

Re: [openstreetmap/openstreetmap-website] Use resourceful routes for user new/create actions (PR #5459)

2025-01-07 Thread Tom Hughes via rails-dev
Merged #5459 into master. -- Reply to this email directly or view it on GitHub: https://github.com/openstreetmap/openstreetmap-website/pull/5459#event-15843185350 You are receiving this because you are subscribed to this thread. Message ID: ___ rails-

Re: [openstreetmap/openstreetmap-website] Use resourceful routes for user new/create actions (PR #5459)

2025-01-07 Thread Tom Hughes via rails-dev
Looks good to me, thanks. -- Reply to this email directly or view it on GitHub: https://github.com/openstreetmap/openstreetmap-website/pull/5459#issuecomment-2576031990 You are receiving this because you are subscribed to this thread. Message ID: ___

Re: [openstreetmap/openstreetmap-website] Support unwrapped bbox values in changeset history queries (PR #5473)

2025-01-07 Thread Tom Hughes via rails-dev
Isn't the real issue here that our `wrap` is just wrong when the bounding box spans the whole world, and that if the minimum and maximum longitude are more than 360 degrees apart it should just result 0 to 360 as the wrapped result? -- Reply to this email directly or view it on GitHub: https://

Re: [openstreetmap/openstreetmap-website] Add ElementsController base class (PR #5479)

2025-01-07 Thread Tom Hughes via rails-dev
Looks good to me, thanks. -- Reply to this email directly or view it on GitHub: https://github.com/openstreetmap/openstreetmap-website/pull/5479#issuecomment-2575971934 You are receiving this because you are subscribed to this thread. Message ID: ___

Re: [openstreetmap/openstreetmap-website] Report deactivates_at as block read date (PR #5478)

2025-01-07 Thread Tom Hughes via rails-dev
Looks good to me, thanks. -- Reply to this email directly or view it on GitHub: https://github.com/openstreetmap/openstreetmap-website/pull/5478#issuecomment-2575968776 You are receiving this because you are subscribed to this thread. Message ID: ___

[openstreetmap/openstreetmap-website] Add ElementsController base class (PR #5479)

2025-01-07 Thread Anton Khorev via rails-dev
You can view, comment on, or merge this pull request online at: https://github.com/openstreetmap/openstreetmap-website/pull/5479 -- Commit Summary -- * Add ElementsController base class * Move browse/feature template to elements/show -- File Changes -- A app/controllers/elements_con

[openstreetmap/openstreetmap-website] Report deactivates_at as block read date (PR #5478)

2025-01-07 Thread Anton Khorev via rails-dev
I changed `updated_at` to `deactivates_at` in #5476 and forgot to make the same change on the next line. This can cause block update dates to be reported as block read dates. Those dates are different for blocks that were edited by moderators after they were read. The reason text can still be up

Re: [openstreetmap/openstreetmap-website] Update arguments of data overlay interestingNode() (PR #5477)

2025-01-07 Thread Tom Hughes via rails-dev
Merged #5477 into master. -- Reply to this email directly or view it on GitHub: https://github.com/openstreetmap/openstreetmap-website/pull/5477#event-15835487271 You are receiving this because you are subscribed to this thread. Message ID: ___ rails-

Re: [openstreetmap/openstreetmap-website] Update arguments of data overlay interestingNode() (PR #5477)

2025-01-07 Thread Tom Hughes via rails-dev
Ah sorry I was working on the exact same thing. Good to have the test as well though. -- Reply to this email directly or view it on GitHub: https://github.com/openstreetmap/openstreetmap-website/pull/5477#issuecomment-2574699364 You are receiving this because you are subscribed to this thread.

[openstreetmap/openstreetmap-website] Update arguments of data overlay interestingNode() (PR #5477)

2025-01-07 Thread Anton Khorev via rails-dev
Makes relation node members (for example, bus stops in routes) *interesting* again. Fixes https://github.com/openstreetmap/leaflet-osm/pull/43#issuecomment-2574558124. You can view, comment on, or merge this pull request online at: https://github.com/openstreetmap/openstreetmap-website/pull/5