Kim Burgestrand

Results 97 comments of Kim Burgestrand

Reading this, it seems like it boiled down to: ```ruby class Parent; end class Child < Parent def self.policy_class = ParentPolicy end class NamespacedChild < Parent def self.policy_class = ActiveAdmin::ParentPolicy...

Hi @mattzollinhofer! I'd be interested in hearing your reaction to the following idea: https://github.com/varvet/pundit/discussions/774

Question on this particular issue, are you looking to specifically get the same _instance_ back, or just the same policy lookup behaviour? For the latter, the above comment seems sufficient...

Hi! Sorry about you not receiving a reply at all. 2022 was hectic, so while we did have a brief look the reply slipped our minds. I'm generally positive about...

Me and @dgmstuart had a chat about this, and I believe we both agreed that query methods should be encouraged to return a boolean (as opposed to raising an exception)....

Revisiting this today, and I don't think that having stateful policies as the default recommended approach is a good idea. Our main state keeping today is the policy/scope cache, and...

Knee-jerk reaction is yes! A bit worried about the caching behaviour in this case. We could probably bypass the cache [just we're already doing it in the other methods](https://github.com/varvet/pundit/blob/aabb3441817c23f3ae41ed1efd468411680d0f53/lib/pundit.rb#L80-L84). My...

Now with `Pundit::Context` I believe we could provide namespacing by overriding the `pundit` context in the controller: https://github.com/varvet/pundit/blob/7e59b98a72850a02c0a10ec17fff425cff18d31a/lib/pundit/authorization.rb#L19-L24 There's no explicit implementation for that, but I'm thinking something like: ```...

@dgmstuart I rebased and created an updated `Gemfile.lock` for today's oldest supported ruby version, but I haven't done anything with the JRuby conundrum partly because I'm also having issues installing...

A downside of this, which could be worked around, is that CI would no longer download the most up to date gems of our dependencies since it would only fetch...