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.
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
Add PL grade type and Make levelbuilders and universal instructors able to join any section #45924
Conversation
@@ -1379,8 +1379,8 @@ describe('teacherSectionsRedux', () => { | |||
); | |||
server.respondWith( | |||
'GET', | |||
'/dashboardapi/sections/valid_course_offerings', | |||
successResponse([]) | |||
'/dashboardapi/sections/available_participant_types', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Intentional: Realized there were two calls to valid course offerings and none to available participant types in this test
@@ -52,6 +52,8 @@ module SharedConstants | |||
|
|||
STUDENT_GRADE_LEVELS = %w(K 1 2 3 4 5 6 7 8 9 10 11 12 Other).freeze | |||
|
|||
PL_GRADE_VALUE = 'pl'.freeze |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
special values are concerning. can you help me understand why this is necessary, rather than using other properties of the section to determine whether it is a PL section?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
after offline conversation, it sounds like we will not have logic predicated on this field and it is just needed for RED purposes. I think this would be great to capture in a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
also, having a validation here eliminates my worry about duplicate state getting out of sync 👍
participant_type 'facilitator' | ||
grade 'pl' | ||
login_type 'email' | ||
end |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice cleanup!
Change 1
To help keep the data for sections as clear as possible we are going to set the grade for professional learnign sections to 'pl' behind the scenes.
I discussed this with RED to make sure this was the best direction to support them.
Change 2
Make sure Levelbuilders and Universal Instructors can join any section.
Links
ENG plan
Testing story