Skip to content

finished impl#190

Merged
SamNie2027 merged 38 commits intomainfrom
admin-track-applicants-frontend
Mar 15, 2026
Merged

finished impl#190
SamNie2027 merged 38 commits intomainfrom
admin-track-applicants-frontend

Conversation

@ostepan8
Copy link
Copy Markdown

ℹ️ Issue

Closes

📝 Description

added an availability table to the applicant view so admins can see and edit when an applicant is available for each day of the week. the availability data comes from the pandadoc form as free-text and gets displayed as-is in the table.

  1. added sundayAvailability field to the application entity since the design includes sunday but it was missing from the model, plus a migration for the new column.
  2. created a new PATCH /applications/:appId/availability endpoint with a dto so admins can update individual day availability without touching other fields.
  3. built an AvailabilityTable component that matches the figma design
  4. created an applicant detail page at /applications/:appId that fetches the application data and renders the availability table.
  5. added getApplication and updateAvailability methods to the frontend api client.
  6. added sundayAvailability to the seed data so there's test data to work with locally.
  7. fixed the discipline table migration to use IF NOT EXISTS for the enum type since it was conflicting with the init migration.
  8. registered the Applicant entity in data-source.ts since it was missing and causing the seed to fail.

✔️ Verification

I played around with the functionality and make it look like the design.

Screenshot 2026-02-25 at 5 56 12 PM

@ostepan8 ostepan8 requested a review from SamNie2027 February 25, 2026 23:02
@SamNie2027
Copy link
Copy Markdown
Collaborator

When you write PRs do you mind making the titles more descriptive of the changes in the future?

Copy link
Copy Markdown
Collaborator

@SamNie2027 SamNie2027 left a comment

Choose a reason for hiding this comment

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

As discussed during standup today, please remove the sundayAvailability component from backend and frontend as BHCHP does not open on sundays.

SamNie2027
SamNie2027 previously approved these changes Mar 15, 2026
@ostepan8 ostepan8 dismissed SamNie2027’s stale review March 15, 2026 20:26

The merge-base changed after approval.

@SamNie2027 SamNie2027 self-requested a review March 15, 2026 20:28
SamNie2027
SamNie2027 previously approved these changes Mar 15, 2026
@ostepan8 ostepan8 dismissed SamNie2027’s stale review March 15, 2026 20:31

The merge-base changed after approval.

rayyanmridha
rayyanmridha previously approved these changes Mar 15, 2026
SamNie2027
SamNie2027 previously approved these changes Mar 15, 2026
@ostepan8 ostepan8 dismissed stale reviews from SamNie2027 and rayyanmridha March 15, 2026 20:37

The merge-base changed after approval.

@SamNie2027 SamNie2027 merged commit aee47f1 into main Mar 15, 2026
4 checks passed
@SamNie2027 SamNie2027 deleted the admin-track-applicants-frontend branch March 15, 2026 20:39
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