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.

Sign up for GitHub

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

Jump to bottom

Hide coaches/staff/instructors from coach's enrollment tab, gradebook, grades csv file #255

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Hide coaches/staff/instructors from coach's enrollment tab, gradebook, grades csv file #255

wants to merge 2 commits into from
+171 −6

Conversation

amir-qayyum-khan
Copy link

Background

Fixes #243

What is done in this PR

Studio Updates: None.
LMS Updates:

  • Now showing ccx members in coach enrollment tab exculding staff, instructor and coach on master course.
  • Hid staff, instructors and coach from gradebook
  • Hid staff, instructors and coach from gradebook csv file

@pdpinch @giocalitri

To test:

  • Create a course
  • Add staff, instructors and coaches to the course.
  • Create CCX
  • See enrollment page in coach dashboard their should be no staff and instructors
  • Open gradebook and see list of student make sure their are no staff, instructors and coaches
  • Download gradebook csv and check their should no staff, instructors and coaches of master course

@pdpinch pdpinch assigned pdpinch and unassigned giocalitri and amir-qayyum-khan Mar 8, 2018
@pdpinch pdpinch removed their assignment Nov 21, 2018
@pdpinch
Copy link
Member

pdpinch commented Nov 21, 2018

I wonder if someone else has already added this functionality upstream to edX?

@amir-qayyum-khan
Copy link
Author

I dont think so. This issue still exist on edX master

cc @tasawernawaz

@pdpinch
Copy link
Member

pdpinch commented Nov 26, 2018

Ok, I'll take this over and either close or reassign it.

@pdpinch pdpinch self-assigned this Nov 26, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@pdpinch pdpinch

Labels
CCX Needs Review
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@amir-qayyum-khan @pdpinch @giocalitri