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

[Test failure] TC_INSTRUCTOR_6: issue adding Group Community TA on the course with no divided discussions #373

Open
GlugovGrGlib opened this issue May 29, 2024 · 1 comment
Labels
redwood release testing Affects the upcoming release (attention needed)
Milestone

Comments

@GlugovGrGlib
Copy link
Member

Release

Redwood

Expected behavior

When adding a community TA to a course without cohorts and divided discussions enabled through the membership tab in the instructor dashboard, the Instructor should see an error, and the user shouldn't be assigned with role community TA.

Actual behavior

The error is displayed as expected, however:

  1. User isn't shown in the community TA section in Membership, but it's seen in the discussions that user has Community TA status.
image
  1. The user is added as Community TA to discussions
image
  1. It isn't possible to revoke access until the divided discussions functionality is enabled for a course.

Steps to reproduce

  • Setup discussions in the course without enabling Divide discussions by cohorts option
  • Try to add user as Community TA on the Membership tab in the Instructor dashboard
  • Check discussions post by the assigned user

Additional information

After the Cohorts are enabled for the course and Divide discussions by cohorts option is enabled in discussions, the user appears in the list of Community TA

image
@mariajgrimaldi
Copy link
Member

Thanks for the report! I can confirm this is currently happening in the Redwood sandbox. I also tried it in Quince, and although the discussion configs differ a bit, I replicated it there.

I created an issue in the edx platform since that's where the instructor dashboard logic lives.

@mariajgrimaldi mariajgrimaldi added this to the Redwood.3 milestone Sep 14, 2024
@mariajgrimaldi mariajgrimaldi modified the milestones: Redwood.3, Sumac.1 Oct 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
redwood release testing Affects the upcoming release (attention needed)
Projects
Status: Backlog
Development

No branches or pull requests

2 participants