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.

Sign up for GitHub

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

Jump to bottom

Class group error #637

Closed
internot169 opened this issue Nov 17, 2023 · 3 comments · Fixed by #640
Closed

Class group error #637

internot169 opened this issue Nov 17, 2023 · 3 comments · Fixed by #640

Comments

@internot169
Copy link
Collaborator

My advanced topics class E period with Kaminsky has way too many students. most of whom are much too young to likely be taking the class.

Submitted via Form by osteinglass@eastsideprep.org
@internot169
Copy link
Collaborator Author

internot169 commented Nov 27, 2023 •
edited
Loading

Strange. Can you please provide a screenshot, so we can verify the data for those users?

sent in email to owen

@internot169
Copy link
Collaborator Author

image

@internot169
Copy link
Collaborator Author

Seems like Dr. Kaminsky is supposedly teaching a study hall at the same time as this Advanced Topics class, so several Middle Schoolers are displayed as part of the class. We do not check by class name if the classes are the same at this moment. Linking pull request to fix.

@internot169 internot169 mentioned this issue Nov 29, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix Class Group Error EastsidePreparatorySchool/epschedule
1 participant
@internot169