feat: grant audit learners access #1180
Conversation
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## master #1180 +/- ##
=======================================
Coverage 87.85% 87.85%
=======================================
Files 263 263
Lines 4480 4481 +1
Branches 1132 1132
=======================================
+ Hits 3936 3937 +1
Misses 530 530
Partials 14 14
☔ View full report in Codecov by Sentry. |
| @@ -20,15 +20,22 @@ const Chat = ({ | |||
| 'executive-education', | |||
| ]; | |||
|
|
|||
| const isVerifiedEnrollmentMode = ( | |||
| const AUDIT_MODES = [ | |||
There was a problem hiding this comment.
Nit. My only comment is there may be a mismatch between these modes and those on the backend.
I noticed that a paid-bootcamp mode is missing from VERIFIED_MODES. I also noticed that there are three Executive Education modes: paid-executive-education, unpaid-executive-education, and executive-education, and we're missing the paid-executive-education mode from VERIFIED_MODES. I don't know if paid-executive-education and executive-education resolve to the same mode at some point (e.g. executive-education), but it's something I noticed.
This isn't an issue for our release right now because of the modes available in the alpha courses, and nothing catastrophic will happen as a result of the mismatch, but I think it's worth taking a quick look at.
There was a problem hiding this comment.
Let me take a closer look to make sure it matches up
There was a problem hiding this comment.
Just updated VERIFIED_MODES to include both paid-bootcamp and paid-executive-education
alangsto
force-pushed
the
alangsto/update_audit_access
branch
from
5146969 to
41084d3
Compare
MST-2130
The learning assistant should be visible to audit learners in courses for which the assistant has been enabled.