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

Resolve assigning administrators to school #117

Open
AmonDeShir opened this issue Nov 12, 2024 · 0 comments
Open

Resolve assigning administrators to school #117

AmonDeShir opened this issue Nov 12, 2024 · 0 comments
Assignees
Milestone

Comments

@AmonDeShir
Copy link
Collaborator

Currently, every user, including administrators, is assigned a school. This creates a problem because the school with ID 1, created by the seeder for the super admin, appears as an option during user registration. This results in a misleading choice for users, as they may select a school that doesn’t actually exist. Additionally, this school may show up in rankings and in the school CRUD page, where administrators could accidentally delete it, potentially removing all assigned administrators.

To resolve this, we need to either restructure the user system by separating students and administrators into different models, or reserve school with ID 1 (created by the seeder) only for administrators. This school could be marked as "disabled" and automatically assigned when creating new administrator accounts.

@AmonDeShir AmonDeShir added this to the Demo v2 milestone Nov 12, 2024
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

No branches or pull requests

2 participants