[openstreetmap/openstreetmap-website] Add a page to change legacy trace visibility in bulk (PR #7368)
Ruben L. Mendoza
notifications at github.com
Wed Sep 9 13:04:15 UTC 2026
@Rub21 commented on this pull request.
> + # one trace more than fits on a single page
+ 2.times { create_list(:trace, 10, :without_validations, :visibility => "public", :user => user) }
+ newest = create(:trace, :without_validations, :visibility => "public", :user => user)
+ ids = user.traces.order(:id => :desc).pluck(:id)
+
+ session_for(user)
+ get edit_traces_legacy_visibility_path
+ assert_response :success
+ assert_select "table#trace_list tbody tr", :count => 20
+ assert_select "table#trace_list tbody tr a", newest.name
+ assert_select "a", :text => "Older Traces", :count => 2
+
+ get edit_traces_legacy_visibility_path(:before => ids[19])
+ assert_response :success
+ assert_select "table#trace_list tbody tr", :count => 1
+ assert_select "a", :text => "Newer Traces", :count => 2
Added the same for `Older Traces`
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/7368#discussion_r3968697670
You are receiving this because you are subscribed to this thread.
Message ID: <openstreetmap/openstreetmap-website/pull/7368/review/5154652276 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20260909/47c62912/attachment.htm>
More information about the rails-dev
mailing list