[openstreetmap/openstreetmap-website] Allow users to delete their own diary entries (PR #4540)

Tom Hughes notifications at github.com
Wed Feb 28 14:34:41 UTC 2024


@tomhughes commented on this pull request.



> @@ -43,7 +43,7 @@ def initialize(user)
         can [:new, :show, :create, :destroy], :oauth2_authorization
         can [:edit, :update, :destroy], :account
         can [:show], :dashboard
-        can [:new, :create, :edit, :update, :comment, :subscribe, :unsubscribe], DiaryEntry
+        can [:new, :create, :edit, :update, :comment, :subscribe, :unsubscribe, :hide, :unhide], DiaryEntry

Sure there is a whole discussion at https://github.com/CanCanCommunity/cancancan/blob/develop/docs/fetching_records.md which suggests it can work but I haven't managed to make any sense of it yet and we're not using it anywhere else so I think we should be consistent here and look at that separately?

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

Message ID: <openstreetmap/openstreetmap-website/pull/4540/review/1906435400 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20240228/ccb99e86/attachment.htm>


More information about the rails-dev mailing list