Skip to content

Adds staff and instructor users of the master course to CCX - #11680

Closed
amir-qayyum-khan wants to merge 1 commit into
openedx:masterfrom
mitocw:enhancement/aq/add_master_course_staff_in_ccx_fix_migration_issue
Closed

Adds staff and instructor users of the master course to CCX#11680
amir-qayyum-khan wants to merge 1 commit into
openedx:masterfrom
mitocw:enhancement/aq/add_master_course_staff_in_ccx_fix_migration_issue

Conversation

@amir-qayyum-khan

Copy link
Copy Markdown
Contributor

Background

fixes mitocw#126, fixes mitocw#155, fixes mitocw#115 and fixes mitocw#205

This is an updated version of https://github.com/edx/edx-platform/pull/10877 which had to be reverted.

What is done in this PR

Studio Updates:

  • Remove ccx from staff dashboard inside studio. Because a master course can have unlimited ccx, staff's dashboard can be populated with a huge list of ccx along with master course

LMS Updates:

  • This is for course staff to see what ccx coaches are doing with course.
  • Now when a coach creates a new ccx, I am assigning staff role on this ccx course to all staff of master course.
  • Added instructors of master course as instructor(s) in ccx.
  • Also adds the master course staff users to new CCX created via the CCX REST APIs
  • Note: This code will add/enrol staff and instructors in ccx and now max student limit for ccx depends on number of staff and admins.
  • Fixed view as student issue on ccx, now staff can view as student and see how ccx view will appear to student
  • Fixed display name on ccx coach dashboard. Previously it was show display name of master course on coach dashboard.

@pdpinch @giocalitri

  • Staff on Master course
    screen shot 2015-11-30 at 5 12 54 pm
  • Staff on CCX
    screen shot 2015-11-30 at 5 13 04 pm
  • Admin/instructors on master course
    screen shot 2015-12-10 at 5 10 18 pm
  • Admin/instructors on ccx
    screen shot 2015-12-10 at 5 10 22 pm

Fixed view as student

screen shot 2015-12-17 at 7 21 37 pm 2
screen shot 2015-12-17 at 7 21 53 pm 2
screen shot 2015-12-17 at 7 22 05 pm 2

Fixed display name of ccx on coach dashboard.

screen shot 2015-12-18 at 3 29 34 pm
screen shot 2015-12-18 at 3 29 44 pm
screen shot 2015-12-18 at 3 29 55 pm

@amir-qayyum-khan
amir-qayyum-khan force-pushed the enhancement/aq/add_master_course_staff_in_ccx_fix_migration_issue branch 3 times, most recently from 8d76a6d to 0a48243 Compare February 29, 2016 13:08
@amir-qayyum-khan amir-qayyum-khan changed the title WIP Staff and instructor users of the master course are added to CCX created via REST APIs and LMS , fixed view as student issue on ccx course and fixed ccx display name on coach dashboard plus fixed some minor issues related to ccx and data migration in ccx Staff and instructor users of the master course are added to CCX created via REST APIs and LMS , fixed view as student issue on ccx course and fixed ccx display name on coach dashboard plus fixed some minor issues related to ccx and data migration in ccx Feb 29, 2016
@pdpinch pdpinch changed the title Staff and instructor users of the master course are added to CCX created via REST APIs and LMS , fixed view as student issue on ccx course and fixed ccx display name on coach dashboard plus fixed some minor issues related to ccx and data migration in ccx Adds staff and instructor users of the master course to CCX Feb 29, 2016
@pdpinch

pdpinch commented Feb 29, 2016

Copy link
Copy Markdown
Contributor

I updated the cover letter a bit.

@amir-qayyum-khan
amir-qayyum-khan force-pushed the enhancement/aq/add_master_course_staff_in_ccx_fix_migration_issue branch from 0a48243 to d4c24a3 Compare February 29, 2016 13:38
@amir-qayyum-khan
amir-qayyum-khan force-pushed the enhancement/aq/add_master_course_staff_in_ccx_fix_migration_issue branch from d4c24a3 to 2140783 Compare February 29, 2016 14:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

2 participants