Skip to content

Implement public cohort - #19877

Merged
nedbat merged 1 commit into
open-release/ironwood.masterfrom
ormsbee/cp_anonymous_cohort
Feb 25, 2019
Merged

Implement public cohort#19877
nedbat merged 1 commit into
open-release/ironwood.masterfrom
ormsbee/cp_anonymous_cohort

Conversation

@ormsbee

@ormsbee ormsbee commented Feb 25, 2019

Copy link
Copy Markdown
Contributor

This PR is based on #19284 and is part of the
series of work related to the proposal #18134.

This PR avoids the assignment of
anonymous/unenrolled users to any cohort when
course is public. Anonymous or unenrolled users
will only see content that does not have a
content group assigned.
The "View Course" link to the course outline
is shown on the course about page for a course
marked public/public outline.
It also makes course handouts available for
public courses (not for public_outline).
This PR also hides the different warnings and
messages asking the user to sign-in and enroll
in the course, when the course is marked public.
It modifies the default public_view text to
include the component display_name when
unenrolled access is not available.

This PR is based on #19284 and is part of the
series of work related to the proposal #18134.

This PR avoids the assignment of
anonymous/unenrolled users to any cohort when
course is public. Anonymous or unenrolled users
will only see content that does not have a
content group assigned.
The "View Course" link to the course outline
is shown on the course about page for a course
marked public/public outline.
It also makes course handouts available for
public courses (not for public_outline).
This PR also hides the different warnings and
messages asking the user to sign-in and enroll
in the course, when the course is marked public.
It modifies the default public_view text to
include the component display_name when
unenrolled access is not available.
@ormsbee

ormsbee commented Feb 25, 2019

Copy link
Copy Markdown
Contributor Author

jenkins run all

@nedbat
nedbat merged commit 1118782 into open-release/ironwood.master Feb 25, 2019
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.

3 participants