Skip to content

[INTF25] Update database to support conflict reporting#71

Merged
mxc-maggiechen merged 1 commit intomainfrom
INTf25-conflict-reporting-db-support
Oct 18, 2025
Merged

[INTF25] Update database to support conflict reporting#71
mxc-maggiechen merged 1 commit intomainfrom
INTf25-conflict-reporting-db-support

Conversation

@SaqAsh
Copy link
Collaborator

@SaqAsh SaqAsh commented Oct 4, 2025

Notion ticket link

Update Database to Support Conflict Reporting

Implementation description

  • Added reviewerHasConflict boolean field to the ReviewedApplicantRecord model with default false
  • Created migration to add this column to the Postgres schema

Steps to test

  1. Run docker exec recruitment_tools_backend node migrate up
  2. Verify that the reviewed_applicant_records table now includes the reviewerHasConflict column (type: boolean, default: false)

What should reviewers focus on?

  • Correctness of the migration script

Checklist

  • My PR name is descriptive and in imperative tense
  • My commit messages are descriptive and in present tense
  • I have run the appropriate linter(s)
  • I have requested a review from the PL and relevant devs

@SaqAsh SaqAsh requested a review from mxc-maggiechen October 4, 2025 17:56
@SaqAsh SaqAsh self-assigned this Oct 4, 2025
@ericahan22 ericahan22 self-requested a review October 4, 2025 18:19
Copy link
Contributor

@mxc-maggiechen mxc-maggiechen left a comment

Choose a reason for hiding this comment

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

LGTM!

@mxc-maggiechen mxc-maggiechen merged commit 76954ca into main Oct 18, 2025
1 check passed
@mxc-maggiechen mxc-maggiechen deleted the INTf25-conflict-reporting-db-support branch October 18, 2025 17:16
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.

3 participants