Skip to content
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

Create available_participant_types endpoint #45579

Merged
merged 2 commits into from Mar 31, 2022

Conversation

dmcavoy
Copy link
Contributor

@dmcavoy dmcavoy commented Mar 31, 2022

Creates a new endpoint for getting data to the create sections flow on the front end. available_participant_types provides an array of the allowed participant_types for a section for a specific user. Users should only be able to create sections with participant_types "below their level". For example teachers can only create sections for students and facilitators can only create sections for students and teachers. This end point will be used to update the section creation flow to allow users with more than one available participant type to pick which participant type the section is for.

Links

ENG Plan
Product Spec

Testing story

  • Added and updated unit tests

@dmcavoy dmcavoy requested a review from a team March 31, 2022 16:06
@@ -388,7 +395,8 @@ export const asyncLoadSectionData = id => dispatch => {
'/dashboardapi/sections',
`/dashboardapi/courses`,
'/dashboardapi/sections/valid_scripts',
'/dashboardapi/sections/valid_course_offerings'
'/dashboardapi/sections/valid_course_offerings',
'/dashboardapi/sections/available_participant_types'
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice job finding a way to make this request in parallel with the other requests 👍

@dmcavoy dmcavoy merged commit 40fd631 into staging Mar 31, 2022
@dmcavoy dmcavoy deleted the eptools/dani/available_participant_types branch March 31, 2022 18:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants