Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: add permissions roles to new course #617

Closed
wants to merge 1 commit into from

Conversation

steff456
Copy link

@steff456 steff456 commented Jan 10, 2024

Private-ref: BB-8292

This PR,

  • Adds permission roles to new course when cloned

This avoids having user roles errors when trying to set Discussion Moderators or Administrators.

Testing instructions

  • Checkout to this branch, git checkout steff456/add-roles-clones
  • Use the API to clone a course
  • Open Studio and go to the instructor tab
  • Try to add a new user under the course team management as a Discussion Moderator or Discussion Administrator
  • No error should appear!

@steff456 steff456 requested a review from kaustavb12 January 10, 2024 16:20
@steff456 steff456 self-assigned this Jan 10, 2024
@steff456
Copy link
Author

I'm closing this PR in favor of https://gitlab.com/opencraft/dev/clone_course/-/merge_requests/7

@steff456 steff456 closed this Jan 12, 2024
@Agrendalath Agrendalath deleted the steff456/add-roles-clones branch January 15, 2024 16:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant