Skip to content

Graduation management first page#1679

Open
Meatchema wants to merge 16 commits intodevelopmentfrom
graduation-management-first-page
Open

Graduation management first page#1679
Meatchema wants to merge 16 commits intodevelopmentfrom
graduation-management-first-page

Conversation

@Meatchema
Copy link
Contributor

@Meatchema Meatchema commented Feb 19, 2026

Issue Description

-The problem is that when the graduation check is clicked it says that the status has been confirmed and it works to create the graduated individual. But, if you uncheck their graduation status, they are no longer searchable under the ‘Senior' Status.

Fixes #1679

Changes

  • Added remove class 'hidden' and redraw table code so that status could go back to senior after being unchecked as graduated

Testing

-set in database/reset_database.sh from-backup

  • Click Admin, then 'Graduation Management'
  • Use bullet points to provide a concise description of testing procedure.
  • Next you will chose a student, I usually use 'Monica' who is a senior
  • With the graduate toggle off (in its white form), click on Monicas grad cap check box (her name should disappear)
  • With the toggle still off, try to search her name in the search bar. You shouldn't be able to find her now that she is Alumni status
  • Next, switch the toggle on and search up Monica. (She should show up and her status should be Alumni)
  • Now, with the toggle still on, unclick Monica's grad cap button (She should still be visible since this version of the list includes both graduates and enrolled but now her status should say senior)
image

@Meatchema Meatchema self-assigned this Feb 19, 2026
@Arohasina Arohasina self-requested a review February 19, 2026 20:26
Copy link
Contributor

@Arohasina Arohasina left a comment

Choose a reason for hiding this comment

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

-tested the changes, all work
-ran ./tests/run_tests.sh everything passed

All good on my side. If the second reviewer approves i believe this PR can be merged

@github-actions
Copy link

View Code Coverage

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

Comments