Skip to content

Conversation

@yifeifang11
Copy link
Collaborator

No description provided.

@yifeifang11 yifeifang11 requested a review from Copilot October 2, 2025 16:01
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, titled "Chungus", makes numerous functional improvements and bug fixes to the fostersource platform. The changes focus on improving course registration management, UI enhancements, and administrative functionality.

Key changes include:

  • Added registration limit functionality to courses with capacity checking
  • Enhanced catalog filtering and registration flow for non-logged-in users
  • Improved admin interfaces with better data handling and UX updates

Reviewed Changes

Copilot reviewed 43 out of 45 changed files in this pull request and generated 6 comments.

Show a summary per file
File Description
frontend/src/store/useCourseEditStore.ts Added registrationLimit field and improved date object hydration
frontend/src/shared/types/course.tsx Added registrationLimit property to Course type
frontend/src/routes/RoutesAndLayouts.tsx Reorganized routes and moved catalog to public access
frontend/src/pages/Catalog/Catalog.tsx Enhanced filtering logic and added isLoggedIn support
frontend/src/pages/Catalog/CatalogCourseComponent.tsx Added capacity checking and login-to-register functionality
frontend/src/pages/Admin/Pricing/Pricing.tsx Replaced hardcoded dropdown with dynamic registration limit input and date pickers
frontend/src/pages/Dashboard/dashboard.tsx Added null course filtering for deleted courses
backend/models/courseModel.ts Added registrationLimit field to course schema
backend/controllers/userController.ts Enhanced user filtering with _id parameter support
Files not reviewed (1)
  • frontend/package-lock.json: Language not supported

@yifeifang11 yifeifang11 merged commit 436b937 into main Oct 2, 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