[openstreetmap/openstreetmap-website] Load notes with remote control (PR #3718)
Anton Khorev
notifications at github.com
Sat Sep 24 21:37:03 UTC 2022
Does part of #957: loads a note when running a remote control edit from `/note/*id*` page.
Normally you do remote control with `load_and_zoom` rc command, and it doesn't support notes. But it's not a problem because you can run another rc command after that. Notes can be loaded by `import` command with a url parameter pointing to note data.
You can view, comment on, or merge this pull request online at:
https://github.com/openstreetmap/openstreetmap-website/pull/3718
-- Commit Summary --
* Treat notes as map objects
* Change note id passing
* Show hidden notes as closed
* Write edit links for notes
* Load note with remote control
-- File Changes --
M app/assets/javascripts/application.js (1)
M app/assets/javascripts/index.js (54)
M app/assets/javascripts/index/note.js (73)
M app/assets/javascripts/leaflet.map.js (84)
M app/assets/javascripts/osm.js.erb (5)
M app/views/browse/note.html.erb (10)
A test/system/browse_test.rb (17)
-- Patch Links --
https://github.com/openstreetmap/openstreetmap-website/pull/3718.patch
https://github.com/openstreetmap/openstreetmap-website/pull/3718.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/3718
You are receiving this because you are subscribed to this thread.
Message ID: <openstreetmap/openstreetmap-website/pull/3718 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20220924/67c09afa/attachment.htm>
More information about the rails-dev
mailing list