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

Tom Hughes notifications at github.com
Thu Nov 1 15:21:41 UTC 2018


tomhughes commented on this pull request.



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

I did do some googling before and there seems to be approximately no consensus on where to put things that aren't one of the "standard" types of class.

I'd always seen `lib` as being for random bits of non-classifiable code and not just for external things (which wouldn't normally be in the repo at all) and in the early days when `lib` was on the autoload path I think that was a more common view but then it got removed.

I didn't even know could just create random directories under `app` to be honest - how does that work? Does rails add every directory there to the autoload 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_r230074245
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20181101/96eed525/attachment-0001.html>


More information about the rails-dev mailing list