[openstreetmap/openstreetmap-website] Use a reduced priority for batch trace uploads (PR #4537)

Tom Hughes notifications at github.com
Sat Feb 24 17:07:36 UTC 2024


This sets the priority of the trace import job based on the number of pending traces the user has so the more they up load at once the lower the priority, thus allowing other users who only upload a few to jump in front of them.
You can view, comment on, or merge this pull request online at:

  https://github.com/openstreetmap/openstreetmap-website/pull/4537

-- Commit Summary --

  * Rename path for changeset API subscription methods
  * Move mechanics of changeset subscriptions into the model
  * Add unsubscribe link to changeset notification mails
  * Add standard List-XXX headers to some notification messages
  * Move scheduling of trace import and destruction to the model
  * Reduce priority of multiple simulataneous trace imports

-- File Changes --

    M app/abilities/ability.rb (1)
    M app/controllers/api/changesets_controller.rb (10)
    M app/controllers/api/traces_controller.rb (4)
    M app/controllers/changesets_controller.rb (29)
    M app/controllers/traces_controller.rb (4)
    M app/mailers/user_mailer.rb (30)
    M app/models/changeset.rb (12)
    M app/models/trace.rb (8)
    M app/views/browse/changeset.html.erb (4)
    A app/views/changesets/_heading.html.erb (15)
    A app/views/changesets/no_such_entry.html.erb (5)
    A app/views/changesets/subscribe.html.erb (12)
    A app/views/changesets/unsubscribe.html.erb (12)
    M app/views/user_mailer/changeset_comment_notification.html.erb (2)
    M app/views/user_mailer/changeset_comment_notification.text.erb (2)
    M config/locales/en.yml (20)
    M config/routes.rb (6)
    M test/controllers/api/changesets_controller_test.rb (20)
    M test/controllers/changesets_controller_test.rb (133)
    M test/models/changeset_test.rb (13)

-- Patch Links --

https://github.com/openstreetmap/openstreetmap-website/pull/4537.patch
https://github.com/openstreetmap/openstreetmap-website/pull/4537.diff

-- 
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/4537
You are receiving this because you are subscribed to this thread.

Message ID: <openstreetmap/openstreetmap-website/pull/4537 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20240224/54e17615/attachment.htm>


More information about the rails-dev mailing list