Closed dhasegan closed 9 years ago
@dhasegan I made some small comments. The only thing I wasn't clear about was whether we actually want to disable the dashboard and the schedule. I think it would give users a reason to log in. Thoughts @akaraj ?
Otherwise LGTM. Pretty nice work on deciders :+1:
Wrote a
decider.py
that basically reads from the deciders.json set in the academy folder and decides if a feature is turned on or off. It is supposed to work with whitelisted users as well. I tested the added module with a pretty test: you can run the test with:./manage.py test app.decider.test
I added a context_processor to always have the available_features in the context and I created a decorator so that we integrate easily with the views so that it fails if you try to load an unimplement feature