li/ednx/BC-14 Add org roles #574
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
magajh
changed the title
magajh
force-pushed
the
li/ednx/BC-14
branch
2 times, most recently
from
b5b2709 to
6c5eeaa
Compare
magajh
requested review from
MaferMazu,
MoisesGSalas,
black-redoc,
jignaciopm and
mariajgrimaldi
black-redoc
reviewed
Nov 4, 2021
Contributor
black-redoc
left a comment
black-redoc
left a comment
There was a problem hiding this comment.
I have let you some comments
MoisesGSalas
approved these changes
Nov 4, 2021
MaferMazu
approved these changes
Nov 5, 2021
mariajgrimaldi
approved these changes
Nov 5, 2021
black-redoc
approved these changes
Nov 5, 2021
black-redoc
approved these changes
Nov 5, 2021
This commit includes: -Refactor course_creator_permissions -calculating rerun permissions for archived courses -Support for custom permissions on course create and rerun handlers -Activating organizations app and fixing tests
11 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
6 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This commit includes:
Commit from previous migration: fa49184
Documentation: https://docs.google.com/document/d/1rIITjfQTGaVQE6rZYYcyuCxRU8QOyvxAyFPE5hSuNaw/edit?pli=1#heading=h.txmiajiyqvs8
Note: when testing this feature in local make sure to collect the studio assets, and after that, login to studio with your test user from an incognito window. By doing this, we make sure the page is loading the updated
CourseOrLibraryListing.jsfile and is able to display the re-run course option.