search

tjcsl / ion

TJ Intranet 3
https://ion.tjhsst.edu
GNU General Public License v2.0
100 stars 91 forks source link

Implement dynamic groups, and merge class groups into them #55

Closed jwoglom closed 9 years ago

jwoglom commented 9 years ago

Dynamic groups would be able to be created based on certain criteria, like in Iodine. If that is implemented, then class groups could just be a subset of those with "user.grade=[9-12]".

ethanal commented 9 years ago

In iodine, aren't dynamic groups used just for classes? True dynamic groups would significantly complicate and slow down any group membership query.

jwoglom commented 9 years ago

They're used for classes, gender, and all student/faculty groups.

On Sun, Mar 29, 2015 at 10:25 AM Ethan Lowman notifications@github.com wrote:

In iodine, aren't dynamic groups used just for classes? True dynamic groups would significantly complicate and slow down any group membership query.

— Reply to this email directly or view it on GitHub https://github.com/tjcsl/ion/issues/55#issuecomment-87420999.

ethanal commented 9 years ago

News posts are never targeted to staff and I don't recall them ever being targeted towards a specific sex. I think it's just too much overhead for a very rare situation. Groups based on only class or staff can be done in sql but everything else requires 1 ldap query per student.