[openstreetmap/openstreetmap-website] Allow to edit revoked blocks (PR #5016)

[Tom Hughes]

@tomhughes commented on this pull request.



> +    assert_redirected_to edit_user_block_path(block)
+    assert_equal "Only the moderators who created or revoked this block can edit it.", flash[:error]
+    block.reload
+    assert_not_predicate block, :active?
+    assert_equal "Original Reason", block.reason
+
+    session_for(creator_user)
+    check_block_updates(block)
+
+    session_for(revoker_user)
+    check_block_updates(block)
+  end
+
+  private
+
+  def check_block_updates(block)

Oh I hadn't realised you had done the same thing in that other PR that I merged yesterday... Personally I would prefer to move that as well.

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

Message ID: <openstreetmap/openstreetmap-website/pull/5016/review/2199808411 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20240725/bdfc03b7/attachment.htm>