Re: [openstreetmap/openstreetmap-website] Documenting API using Swagger / rswag (#3107)

2025-04-24 Thread Anton Khorev via rails-dev
AntonKhorev left a comment (openstreetmap/openstreetmap-website#3107) Is [that post](https://github.com/openstreetmap/openstreetmap-website/issues/3107#issuecomment-2828915771) supposed to be read after chatgpt is mentioned? What is 50% implemented? We don't even have a complete api implementat

Re: [openstreetmap/openstreetmap-website] Add PWA geo protocol handler (PR #5736)

2025-04-24 Thread Anton Khorev via rails-dev
@AntonKhorev commented on this pull request. > @@ -121,6 +135,27 @@ OSM = { return mapParams; }, + parseGeoURI: function (geoURI) { +if (!geoURI || !URL.canParse(geoURI)) return; +const url = new URL(geoURI); +if (url.protocol !== "geo:" || !url.pathname) return; +cons

[openstreetmap/openstreetmap-website] Bump osm-community-index from 5.9.1 to 5.9.2 (PR #5948)

2025-04-24 Thread dependabot[bot] via rails-dev
Bumps [osm-community-index](https://github.com/osmlab/osm-community-index) from 5.9.1 to 5.9.2.
Release notes

Sourced from osm-community-index's releases.

v5.9.2


[openstreetmap/openstreetmap-website] Bump leaflet.locatecontrol from 0.84.1 to 0.84.2 (PR #5949)

Bumps [leaflet.locatecontrol](https://github.com/domoritz/leaflet-locatecontrol) from 0.84.1 to 0.84.2.
Commits

Re: [openstreetmap/openstreetmap-website] Documenting API using Swagger / rswag (#3107)

tomhughes left a comment (openstreetmap/openstreetmap-website#3107) > you're integrating Swagger by writing **RSpec tests that double as > documentation** So only the one small problem then - we don't use RSpec. -- Reply to this email directly or view it on GitHub: https://github.com/openstree

Re: [openstreetmap/openstreetmap-website] Allow getting a GPX in the browser outside openstreetmap.org (Issue #5639)

HarelM left a comment (openstreetmap/openstreetmap-website#5639) Any updates on this? Is there anyway I can help? I'm removing part of my backend that are only a passthrough to OSM API and this is still a problem unfortunately when running from the browser... -- Reply to this email directly or

Re: [openstreetmap/openstreetmap-website] Add dir=auto to user generated content (PR #3429)

NeatNit left a comment (openstreetmap/openstreetmap-website#3429) Thank you @tomhughes for merging #5840 today! Let me know if there's any way I could help with this one. In particular, there are small things I've missed in my PRs because I didn't have the capacity to test properly. Namely: -

Re: [openstreetmap/openstreetmap-website] Documenting API using Swagger / rswag (#3107)

HarelM left a comment (openstreetmap/openstreetmap-website#3107) +1 for swagger/openAPI generated docs instead of a wiki maintenance, see my comment here: https://github.com/openstreetmap/openstreetmap-website/pull/5943. In general, now that OSM backend supports Oauth2 token, one can set this to

Re: [openstreetmap/openstreetmap-website] Add support for json traces API (PR #5943)

mmd-osm left a comment (openstreetmap/openstreetmap-website#5943) Thanks! By the way, I've proposed swagger / openapi a few years ago in #3107... -- Reply to this email directly or view it on GitHub: https://github.com/openstreetmap/openstreetmap-website/pull/5943#issuecomment-2828879487 You are

Re: [openstreetmap/openstreetmap-website] Add support for json traces API (PR #5943)

HarelM left a comment (openstreetmap/openstreetmap-website#5943) I wanted to see that it is deployed first, but sure, I've updated the wiki page. As a side note, wouldn't it be easier to simply generate an openAPI/swagger page that is generated from the code itself to avoid mismatches? -- Repl

Re: [openstreetmap/openstreetmap-website] Add support for json traces API (PR #5943)

mmd-osm left a comment (openstreetmap/openstreetmap-website#5943) @HarelM : It would be nice if you could document your API change on the API 0.6 wiki page ( https://wiki.openstreetmap.org/wiki/API_v0.6 ) to help others developers. You can find a bit of change history in the header section. Fur

Re: [openstreetmap/openstreetmap-website] Add dir="auto" to Markdown content for bidi (PR #5840)

Merged #5840 into master. -- Reply to this email directly or view it on GitHub: https://github.com/openstreetmap/openstreetmap-website/pull/5840#event-17395360665 You are receiving this because you are subscribed to this thread. Message ID: ___ rails-

Re: [openstreetmap/openstreetmap-website] Add dir="auto" to Markdown content for bidi (PR #5840)

tomhughes left a comment (openstreetmap/openstreetmap-website#5840) This has been going for a while now and the latest version looks reasonable to me so I'm going to merge it - if anybody does think there are further improvements possible then they can be addressed in followup PRs. -- Reply to

Re: [openstreetmap/openstreetmap-website] Add support for json response for `/api/0.6/user/gpx_files` endpoint (Issue #5646)

Closed #5646 as completed via #5943. -- Reply to this email directly or view it on GitHub: https://github.com/openstreetmap/openstreetmap-website/issues/5646#event-17395360959 You are receiving this because you are subscribed to this thread. Message ID: _

Re: [openstreetmap/openstreetmap-website] Add support for json traces API (PR #5943)

Merged #5943 into master. -- Reply to this email directly or view it on GitHub: https://github.com/openstreetmap/openstreetmap-website/pull/5943#event-17395360686 You are receiving this because you are subscribed to this thread. Message ID: ___ rails-

Re: [openstreetmap/openstreetmap-website] Add support for json traces API (PR #5943)

@tomhughes approved this pull request. Thanks for all this work - this looks good to me now. I did push a small change to tidy up the commit message and I'll merge it now. -- Reply to this email directly or view it on GitHub: https://github.com/openstreetmap/openstreetmap-website/pull/5943#pu

Re: [openstreetmap/openstreetmap-website] Add support for json traces API (PR #5943)

@tomhughes pushed 1 commit. 4c44df5fdf01c6d4f758c997b7f402fb512af4ca Add support for JSON traces API -- View it on GitHub: https://github.com/openstreetmap/openstreetmap-website/pull/5943/files/43b312166ce3e2e3db5749d5e6398d780376f82f..4c44df5fdf01c6d4f758c997b7f402fb512af4ca You are receiving

Re: [openstreetmap/openstreetmap-website] Add kilometers/miles switch to directions (PR #5915)

@AntonKhorev commented on this pull request. > + let distanceUnits = "km"; let downloadURL = null; - function formatTotalDistance(m) { -if (m < 1000) { - return OSM.i18n.t("javascripts.directions.distance_m", { distance: Math.round(m) }); -} else if (m < 1) { - retu

[openstreetmap/openstreetmap-website] Invisible "X" button when using dark mode preference (Issue #5947)

Dimitar created an issue (openstreetmap/openstreetmap-website#5947) ### URL https://www.openstreetmap.org/node/8043886647 (or any place where an "X" button is visible) ### How to reproduce the issue? 1. Set "Preferred Website Color Scheme" to "Auto" in https://www.openstreetmap.org/prefe

Re: [openstreetmap/openstreetmap-website] Add dir="auto" to Markdown content for bidi (PR #5840)

NeatNit left a comment (openstreetmap/openstreetmap-website#5840) @gravitystorm or @tomhughes Perhaps you could review? -- Reply to this email directly or view it on GitHub: https://github.com/openstreetmap/openstreetmap-website/pull/5840#issuecomment-2827099760 You are receiving this because yo

Re: [openstreetmap/openstreetmap-website] Add yearly template and CalendarLabel plugin for month labels (PR #5906)

@kcne pushed 1 commit. 9b70c849ebce40093dcd92a296f91264f2e7e9b3 do render labeling manually -- View it on GitHub: https://github.com/openstreetmap/openstreetmap-website/pull/5906/files/4d89f0f59451770ef9704e3a19bfacd9fb42a9cf..9b70c849ebce40093dcd92a296f91264f2e7e9b3 You are receiving this beca

Re: [openstreetmap/openstreetmap-website] Add support for json traces API (PR #5943)

@hlfan approved this pull request. -- Reply to this email directly or view it on GitHub: https://github.com/openstreetmap/openstreetmap-website/pull/5943#pullrequestreview-2789959159 You are receiving this because you are subscribed to this thread. Message ID: __