search

lizardsystem / lizard-auth-server

Django backend for the old SSO server
http://lizard-auth-server.readthedocs.io/
MIT License
2 stars 1 forks source link

Demo for a dependent roles feature #26

Closed reinout closed 8 years ago

reinout commented 9 years ago

Ik maak er even een PR van zodat ik er commentaar op kan leveren :-)

reinout commented 9 years ago

Basisidee lijkt me goed.

Waar ik vooral even naar wil kijken is de plaats van de functionalteit en de naamgeving ervan.

Een classmethod die gaat yielden in plaats van een lijst terug te geven: daar wil ik even langer naar kijken. Mijn gok is dat het met een creatieve dikke vette query ook zou moeten kunnen, hoewel die wel leesbaar moet blijven :-)

En dat op de DependentRoles een methode wordt aangeroepen die uiteindelijk geen DependentRoles maar Roles teruggeeft: dat is ook niet helemaal lekker.

Maar goed, daar is het een eerste demo voor! Ik ga er maandag wel even naar kijken. Kijken wat er met naamgeving te doen is. En in m'n achterhoofd vraag ik me af of we met nieuwe django 1.7/1.8 DB features wat leuks kunnen doen.

Het ziet er in ieder geval uit alsof we met jouw idee wel alles kunnen supporten! Elegante oplossing.

reinout commented 8 years ago

Deze zit er op een andere manier al in, dus ik sluit 'em.