[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