[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