Script for granting existing instructors creator access. - #285
Conversation
ghost
assigned
There was a problem hiding this comment.
Maybe we should document that this is to grandfather in existing instructors. Ideally this will only be run once, ever.
There was a problem hiding this comment.
Or perhaps I should just add a task to delete this code (and the django command) at the end of this story?
There was a problem hiding this comment.
Although perhaps we will want to run it on edx at some point. Hmmmmm.
There was a problem hiding this comment.
Right, this kinda feels like a "Migration", but without a schema change. I don't know much about those tools to make a recommendation, maybe Dave O or Victor might have a suggestion.
There was a problem hiding this comment.
All add a task to the story to delete the code if appropriate.
|
+1 after a slight doc string tweek. |
|
+1 thanks for the clarification. |
|
Builds failing, but it seems to be an internal Jenkins/nose error?!? Do you want to trigger another build? |
|
Thanks, Chris-- I hadn't noticed. I triggered a manual build and that was fine, except for pep8 and pylint going up. I did some cleanup and will merge this assuming tests pass. |
Script for granting existing instructors creator access.
Fixed bugs caused by modal cloning. Now, only video modals are cloned.
…lationship ziafazal/api-org-groups-relationship: added groups
added lack of translation on login page(openedx#285)
Adding split mongo fixes back. Not turning split mongo on for new courses, just adding the code that fixes tests and functionality when we do decide to turn it on
LinkedIn button size and single column for view grades button in mobile
2 participants
No description provided.