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

[Andy Allan]

gravitystorm commented on this pull request.



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

Yes, this is where the generator puts it. I'm happy to put it where ever suits - perhaps in config? I'm not sure about lib, I consider that somewhere where stuff goes that (in theory) could be extracted into a gem, but I can see the logic there since we put various classes into lib already.

>From https://github.com/CanCanCommunity/cancancan/blob/develop/lib/generators/cancan/ability/USAGE: 

>The cancan:ability generator creates an Ability class in the models
  directory. You can move this file anywhere you want as long as it
is in the load path.

-- 
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_r225889117
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20181017/163e56e9/attachment.html>