[openstreetmap/openstreetmap-website] Move to CanCanCan for authorization (#2023)

Chris Flipse notifications at github.com
Thu Nov 1 15:07:38 UTC 2018


cflipse commented on this pull request.



> @@ -0,0 +1,57 @@
+# frozen_string_literal: true
+
+class Ability

`/lib` is almost certainly the wrong place, as this isn't external / library code.  I could see putting it in config, but it's active code that gets checked, and config isn't the first place I'd look.  Ben's thought about app/abilities appeals, if the plan is to deconstruct the large object at some point ...



-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/2023#discussion_r230068522
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20181101/4b3d6cae/attachment.html>


More information about the rails-dev mailing list