Skip to content

filter added for rosters to exclude graduated students - #1799

Open
MImran2002 wants to merge 2 commits into
developmentfrom
rosterGraduatedHF
Open

filter added for rosters to exclude graduated students#1799
MImran2002 wants to merge 2 commits into
developmentfrom
rosterGraduatedHF

Conversation

@MImran2002

Copy link
Copy Markdown
Contributor

Issue Description

Fixes issue #1781

  • Students who are marked as graduated in Graduation Management are still appearing in all three rosters including interested student lists throughout CELTS.
  • Graduated students should no longer be included in these lists once their graduation status has been updated just like in Graduation Management page.

Changes

  • Add filter in getProgramInterest so that it only returns students who are either freshman, sophomore, junior or senior
  • Add filter in getParticipantsForProgramForAY so that it only returns students who are either freshman, sophomore, junior or senior. But also a comment was left saying this filter could pose a problem if in the future someone decided to use this function to get participants from programs in older academic year.

Testing

  • Check out the branch git checkout rosterGraduatedHF and git pull
  • Reset the Database to database/reset_database.sh test and run the test tests/run_tests.sh
  • Then switch back to database/reset_database.sh from-backup
  • Navigate to Admin > Settings > Program Management > Click one of the View Rosters and will only see participants still enrolled in the college in all three rosters.

@MImran2002 MImran2002 changed the title graduation filter added for rosters filter added for rosters to exclude graduated students Aug 25, 2026
@MImran2002
MImran2002 requested a review from BrianRamsay August 25, 2026 18:28
@bakobagassas

Copy link
Copy Markdown
Contributor
  • It effectively only displays enrolled students
  • However, when I click on "add an interested student" and input the name of a student that already graduated, it still says the student was added, but it does not add them. To avoid mixed signals, shouldn't we just let them know that the student has graduated and cannot be added, or something like that?

@bakobagassas
bakobagassas self-requested a review August 26, 2026 20:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants