Skip to content

Fe/dashboard fix - #152

Merged
dashaaaa21 merged 5 commits into
developfrom
FE/dashboard-fix
Mar 7, 2026
Merged

dashaaaa21 merged 5 commits into
developfrom
FE/dashboard-fix

Conversation

@dashaaaa21

Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI review requested due to automatic review settings March 7, 2026 17:59
@HackYourFutures
HackYourFutures temporarily deployed to study-bridge-fe-dashboa-ybknih March 7, 2026 18:00 Inactive

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 adds backend validation to prevent students from creating duplicate appointments for the same date/time, and includes small frontend layout/style adjustments to stabilize UI presentation.

Changes:

  • Added a new validation error message and business validation step for duplicate appointments (HTTP 409).
  • Introduced an AppointmentQuery.findExistingAppointment query helper to detect existing active appointments.
  • Simplified auth form styling and constrained review card heights to avoid overflow.

Reviewed changes

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

Show a summary per file
File Description
server/src/validation/appointment/appointmentValidation.constants.ts Adds a new duplicate-appointment validation error message constant.
server/src/validation/appointment/appointmentBusinessValidation.ts Adds duplicate appointment validation and wires it into appointment creation validation flow.
server/src/repositories/queryRepositories/appointment.query.ts Adds a query method to find an existing appointment by student/date/time.
client/src/styles/auth.css Removes animated border styling from the auth form.
client/src/components/homeSection/Reviews/ReviewCardHome.tsx Sets a fixed card height and hides overflow to keep review cards uniform.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

@HackYourFutures
HackYourFutures had a problem deploying to study-bridge-fe-dashboa-ybknih March 7, 2026 18:25 Failure
@HackYourFutures
HackYourFutures temporarily deployed to study-bridge-fe-dashboa-ybknih March 7, 2026 18:40 Inactive
@dashaaaa21
dashaaaa21 merged commit 54a3277 into develop Mar 7, 2026
2 checks passed

This branch was previously deployed

1 inactive deployment
study-bridge-fe-dashboa-ybknih — 6f09be0b Deployed Mar 7, 2026 by HackYourFutures
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.

4 participants