Skip to content

Conversation

@punjarmaan
Copy link

@punjarmaan punjarmaan commented Nov 24, 2025

Summary: Added flagging feature to allow HR/Coordinator to flag hiring assignments on the onboarding page

Changes:

  • Add flag button on each row of hiring assignment table
  • Auto-sort assignments to display flagged applicants first
  • Added toggle button to filter (on FE) through all, flagged, and non-flagged applicants easily
  • Add flagged boolean to hiring assignment entities to store
Screen.Recording.2025-11-20.at.3.53.41.PM.mov

Copy link
Member

@ajaygandecha ajaygandecha left a comment

Choose a reason for hiding this comment

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

Hello team, great work on this! LGTM, very nice work on the design here - I am going to request a final review from @KrisJordan to approve and ensure the migration is setup up correctly, then we can merge.

I added a quick commit for two things:

  1. The optimistic updating logic for the flagging UI works well - but if there is an error, we need to roll back the change. I added quick error handling for this (and a small refactor to add a snackbar alert if an error occurs when updating, which is helpful across to the entire feature)
  2. I switched the hard-coded hex values for the flag icon to use the colors generated by material's CSS functions!

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