Skip to content

Improve student activity registration system#2

Merged
leoglaz merged 3 commits intomainfrom
accelerate-with-copilot
Oct 17, 2025
Merged

Improve student activity registration system#2
leoglaz merged 3 commits intomainfrom
accelerate-with-copilot

Conversation

@leoglaz
Copy link
Owner

@leoglaz leoglaz commented Oct 17, 2025

This pull request adds new features to the Mergington High School Activities API and frontend, including support for unregistering participants, improved activity and participant display, and comprehensive test coverage. The backend now supports both signup and unregister operations with capacity and duplicate checks, while the frontend provides a more interactive and informative UI for managing activity participants. Additionally, a full test suite has been introduced to ensure reliability.

Backend API enhancements:

  • Added an endpoint to unregister a student from an activity, including validation for participant presence and activity existence (src/app.py).
  • Improved signup logic to prevent duplicate signups and enforce activity capacity limits (src/app.py).
  • Expanded the set of activities to include sports, artistic, and intellectual clubs, each with initial participants (src/app.py).

Frontend improvements:

  • Enhanced the activities list to display current participants, including an option to remove (unregister) them directly from the UI (src/static/app.js, src/static/styles.css) [1] [2] [3].
  • Updated the signup process to use POST requests with form data, and ensured the UI refreshes appropriately after signup or unregister actions (src/static/app.js) [1] [2].

Testing and documentation:

  • Added a comprehensive test suite covering all API endpoints, workflows, and edge cases, achieving 100% code coverage (tests/README.md, tests/__init__.py, tests/conftest.py) [1] [2] [3].

@leoglaz leoglaz merged commit 5c87e0a into main Oct 17, 2025
3 checks passed
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.

1 participant