Versions Compared

Key

  • This line was added.
  • This line was removed.
  • Formatting was changed.

[12:15:15 CDT(-0500)] <athena> EricDalquist: when you're around again, i wanted to talk about apetro's admin permissions feature request
[14:43:14 CDT(-0500)] <athena> EricDalquist: i've got a working implementation of the permissions code that recognizes ALL_CHANNELS, ALL_CATEGORIES, and ALL_GROUPS
[14:43:21 CDT(-0500)] <athena> turns out ALL_USERS is actually kinda hard to do
[14:43:22 CDT(-0500)] <EricDalquist> neat
[14:43:42 CDT(-0500)] <athena> not sure though if i should check this in or if we want to replace it with just some sort of ALL_PERMISSIONS, like apetro asked for
[14:43:59 CDT(-0500)] <athena> but maybe it's also useful to be able to permission stuff at a more fine-grained level?
[14:44:10 CDT(-0500)] <EricDalquist> yeah
[14:44:14 CDT(-0500)] <EricDalquist> I think this is reasonable
[14:44:21 CDT(-0500)] <EricDalquist> having 3 top level concepts shouldn't be too much overload
[14:45:38 CDT(-0500)] <athena> might have to just keep an eye on performance, since to tell what kind of target it is we have to try to get it as a group
[14:45:44 CDT(-0500)] <athena> but don't think that'll be the end of the world
[14:46:08 CDT(-0500)] <athena> wish we could do ALL_USERS too, but seems like GroupService.getGroupMember returns an entity whether it exists or not
[14:46:23 CDT(-0500)] <EricDalquist> hrm