Maria Grimaldi (Majo)
Maria Grimaldi (Majo)
I've been thinking hard about this PR. Now, enrollment starts will have a default making courses unavailable for anonymous users after creation, which sounds good, IMO. My issue relies on...
I'll review this again today. Sorry for the delay
In case anyone wants to replicate the testing of this PR, this is what I did. First, I configured my testing environment by doing the following: 1. Ensure FEATURES['ENABLE_COURSE_DISCOVERY'] =...
@jmakowski1123 @mphilbrick211: hello there! before merging this PR, should we notify someone of these changes?
@Ian2012: we need to wait for the product review. Is there anything we could do in the meantime? @jmakowski1123
@Ian2012 can you rebase with the master branch? so we can merge. Also @mphilbrick211, thanks for the ping! This PR still has the product review label, should we remove it?
@mphilbrick211 good! -just checking- this can be merged, right -from the product view-?
@mphilbrick211: the branch is still out-of-date, once @Ian2012 updates it, I'll merge this :)
@mphilbrick211: thank you for picking this up! Regarding your question, yes, that's still the case.
@jmakowski1123: I agree with both of your suggestions. 1. Enabling just the honor course mode ultimately limits the functionality and makes it less valuable for the community. So as you...