Skip to content

Conversation

@Luhfernanda
Copy link
Owner

This pull request introduces several enhancements to the school activities management system, including new activities, improved signup validation, and better participant display functionality. The changes span across backend logic, frontend rendering, and styling.

Backend changes:

  • Added six new activities categorized as sports, artistic, and intellectual activities to the activities dictionary in src/app.py. Each activity includes a description, schedule, maximum participants, and initial participants.
  • Updated the signup_for_activity function in src/app.py to validate that a student is not already signed up for an activity before adding them. This prevents duplicate signups.

Frontend changes:

  • Enhanced the activity cards in src/static/app.js to include a formatted list of participants. If no participants are signed up, a message indicating "Nenhum inscrito ainda" (no participants yet) is displayed.

Styling changes:

  • Added new CSS rules in src/static/styles.css to style the participants section in activity cards, including formatting for participant lists and messages for no participants.

@Luhfernanda Luhfernanda requested a review from Copilot June 27, 2025 14:44
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR enhances the school activities management system by adding new activities, signing up validation to prevent duplicate registrations, and improved participant display in the frontend.

  • Added six new activities in the backend with categorized details
  • Updated the signup logic to check for duplicate registrations
  • Enhanced the frontend activity cards with formatted participant lists and corresponding styling

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
src/static/styles.css New CSS rules to style the participants section in activity cards
src/static/app.js Updated activity cards HTML to include formatted participants info
src/app.py Added new activities and duplicate signup validation in backend

@Luhfernanda Luhfernanda merged commit 3b7c315 into main Jun 27, 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.

2 participants