Re: [openstreetmap/openstreetmap-website] Add short links to OSM objects (Issue #5244)

2024-10-01 Thread Anton Khorev via rails-dev
Users by uid: #4215 -- Reply to this email directly or view it on GitHub: https://github.com/openstreetmap/openstreetmap-website/issues/5244#issuecomment-2387097197 You are receiving this because you are subscribed to this thread. Message ID: ___ rail

Re: [openstreetmap/openstreetmap-website] Add short links to OSM objects (Issue #5244)

2024-10-01 Thread Anton Khorev via rails-dev
`n6444542941`, `w138935876`, `r2389520` format for elements is used in Josm for some things, but I've never seen `t4276888` for notes. -- Reply to this email directly or view it on GitHub: https://github.com/openstreetmap/openstreetmap-website/issues/5244#issuecomment-2387102035 You are receivi

Re: [openstreetmap/openstreetmap-website] Add short links to OSM objects (Issue #5244)

2024-10-01 Thread Tom Hughes via rails-dev
One option would be something like https://osm.org/n/12345678 with single letter n/w/r/c/u as a namespace which as @AntonKhorev has just said also deals with the long requested support for user by uid. The only issue is node/note both wanting the same letter :-( -- Reply to this email directly

Re: [openstreetmap/openstreetmap-website] Masks off unused translation keys in activerecord (PR #5242)

2024-10-01 Thread mmd via rails-dev
As a follow up, this use case is not (yet) supported by i18n-tasks. The referenced GitHub issue has some comments on how this could be implemented. -- Reply to this email directly or view it on GitHub: https://github.com/openstreetmap/openstreetmap-website/pull/5242#issuecomment-2387079370 You a

Re: [openstreetmap/openstreetmap-website] Add short links to OSM objects (Issue #5244)

2024-10-01 Thread mmd via rails-dev
_I'm not covering the question how and where we're generating these links in the first place._ > Links to OSM entities are quite unwieldy, e. g.: > https://www.openstreetmap.org/changeset/157188078. There's still the third option to use `osm.org` along with the current schema: `https://osm.org

Re: [openstreetmap/openstreetmap-website] Email links in text/plain ending in "." - break links (#1844)

2024-10-01 Thread Anton Khorev via rails-dev
Only this option is translation-proof, if the url is wrapped before the substitution: > Reply or learn more about the note at > \. But it's not client-proof, as you can see here on Github. -- Reply to this email directly or view it on GitHub: https:

Re: [openstreetmap/openstreetmap-website] Masks off unused translation keys in activerecord (PR #5242)

2024-10-01 Thread Tom Hughes via rails-dev
Thanks - this looks good to me now. -- Reply to this email directly or view it on GitHub: https://github.com/openstreetmap/openstreetmap-website/pull/5242#issuecomment-2386663309 You are receiving this because you are subscribed to this thread. Message ID: __

Re: [openstreetmap/openstreetmap-website] Masks off unused translation keys in activerecord (PR #5242)

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

[openstreetmap/openstreetmap-website] Add short links to OSM objects (Issue #5244)

2024-10-01 Thread Jörn Heissler via rails-dev
### Problem Links to OSM entities are quite unwieldy, e. g.: . Long links can be problematic in various situations, e.g. QR codes, short messages, emails in a text terminal. ### Description Could you implement short links? For above this might

Re: [openstreetmap/openstreetmap-website] Add short links to OSM objects (Issue #5244)

2024-10-01 Thread Tom Hughes via rails-dev
There's a question about whether this is an operational issue that would be implemented outside of this repository or not - it could be done either way. If it was to be specific to osm.org then it would be an operational matter for sure. Most of what you're suggesting (except notes and users) i

Re: [openstreetmap/openstreetmap-website] Add short links to OSM objects (Issue #5244)

2024-10-01 Thread Tom Hughes via rails-dev
Implementing it in the routing here is certainly one possibility yes - it does mean that if you use osm.org then you will get a double redirect. I don't think we could (or would want to) put it at the top level if it was across all domains, so we'd need something like `/go/` in the existing sho

Re: [openstreetmap/openstreetmap-website] Add short links to OSM objects (Issue #5244)

2024-10-01 Thread Jörn Heissler via rails-dev
> There's a question about whether this is an operational issue that would be > implemented outside of this repository or not - it could be done either way. > If it was to be specific to osm.org then it would be an operational matter > for sure. The same scheme should work for ``www.openstreetm

Re: [openstreetmap/openstreetmap-website] Map Data checkbox: perhaps use toggle slider instead (PR #5009)

2024-10-01 Thread David Tsiklauri via rails-dev
@AntonKhorev I couldn't reproduce JS errors in the console. But I changed logic and now when user clicks "Map Data" checkbox, previous action is skipped immediately. Therefore, there will be no 2*n loads and errors. When we will be okay with the solution, I'll create a new PR in the [leaflet-os

Re: [openstreetmap/openstreetmap-website] Map Data checkbox: perhaps use toggle slider instead (PR #5009)

2024-10-01 Thread David Tsiklauri via rails-dev
@nertc pushed 1 commit. 4b91e799bfed30f71dac5d10723dec016a34c26b Map Data asynchronous checkbox -- View it on GitHub: https://github.com/openstreetmap/openstreetmap-website/pull/5009/files/ca019cd28c25eed1bd211f070b0bd8f10293878d..4b91e799bfed30f71dac5d10723dec016a34c26b You are receiving this