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 interface to allow Admin to mark students as graduated #1315

Open
bledsoef opened this issue Aug 25, 2024 · 1 comment · May be fixed by #1355
Open

Create interface to allow Admin to mark students as graduated #1315

bledsoef opened this issue Aug 25, 2024 · 1 comment · May be fixed by #1355
Assignees

Comments

@bledsoef
Copy link
Contributor

We recently created functionality to allow students to be marked as "graduated" however this is pretty meaningless to us right now because we are unable to actually set someone as graduated. So, we need to create an interface that admin can access and mark students as graduated. I would recommend something kind of like "minor management" that we currently have, but instead there is a list of all students (with the ability to search) where we can check off students as graduated.

Considerations

  1. How can we make this scalable and prevent it from getting super cluttered as the amount of students who are graduated grow?

My suggestion might be to hide students that are graduated and only allow them to searchable, or also implement a toggle like for past events which will show them.

  1. Should we add this to an existing page or create a new sidebar option with something like "Student Management" or "Graduation Management".

Unless we were able to just mark a student as graduated via their profile (which would be kind of inconvenient to do in mass. I would recommend creating a new page, or adding a modal in Settings under User Management.

  1. Should we assume Graduated student cannot attend login and thus we should not have to add any additional logic to deactivate them when they are marked as graduated? How should we adjust their user profile while still leaving it visible?

Would definitely talk to Brian about this before picking it up.

@bledsoef
Copy link
Contributor Author

@Karina-Agliullova Karina-Agliullova self-assigned this Sep 4, 2024
@thealphagurlux thealphagurlux self-assigned this Oct 25, 2024
@thealphagurlux thealphagurlux linked a pull request Oct 25, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants