[openstreetmap/openstreetmap-website] Make loadSidebarContent use promises (PR #7226)
Marwin Hochfelsner
notifications at github.com
Mon Jul 13 22:52:43 UTC 2026
Closes #7225 with a more idiomatic solution.
You can view, comment on, or merge this pull request online at:
https://github.com/openstreetmap/openstreetmap-website/pull/7226
-- Commit Summary --
* Reuse feed selector
* Make XHR parameter addition less clunky
* Make loadSidebarContent use promises
* Add error handling to loadSidebarContent
* Update title only after content download
-- File Changes --
M app/assets/javascripts/index.js (35)
M app/assets/javascripts/index_modules/changeset.js (8)
M app/assets/javascripts/index_modules/directions.js (2)
M app/assets/javascripts/index_modules/element.js (2)
M app/assets/javascripts/index_modules/export.js (3)
M app/assets/javascripts/index_modules/history.js (3)
M app/assets/javascripts/index_modules/new_note.js (5)
M app/assets/javascripts/index_modules/note.js (7)
M app/assets/javascripts/index_modules/query.js (5)
M app/assets/javascripts/index_modules/search.js (3)
-- Patch Links --
https://github.com/openstreetmap/openstreetmap-website/pull/7226.patch
https://github.com/openstreetmap/openstreetmap-website/pull/7226.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/7226
You are receiving this because you are subscribed to this thread.
Message ID: <openstreetmap/openstreetmap-website/pull/7226 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20260713/b6451799/attachment.htm>
More information about the rails-dev
mailing list