Closed hoerstl closed 2 months ago
Before requesting a review for your PR, make sure that the page you were working on or any other related one doesn't break. I couldn't load a Bonner management page, as shown in the image above. Also, 4 checks fail out of 5. One test case fails
Add New Cohorts to Bonner
Issue
Resolves #1133
Currently, a new cohort year is added automatically at a certain date, we should change it to have a button to manually add more cohort years instead of having it happen automatically.
This allows administrators to add people to a cohort by hand as early as they know who should be in it.
Solution
We added a button that adds a new cohort to the Bonner Management page.
This button does not permanently add a cohort but rather makes it visible for the current session so that administrators can add students to it which then makes the cohort persistent.
To Test
Setup
From the
celts
directorygit checkout 1133-Bonnor-Cohort-Control
(yes, I know I spelled Bonner incorrectly in the branch title but in my defense, that's how it should be spelled)database/reset_database.sh from-backup
flask run
Testing