Skip to content

Fix missing availability badge in the add-applicant dropdowns - #177

Merged
adescoteaux1 merged 1 commit into
mainfrom
ally/preference-list-availability-dropdown-fix
Aug 22, 2026
Merged

Fix missing availability badge in the add-applicant dropdowns#177
adescoteaux1 merged 1 commit into
mainfrom
ally/preference-list-availability-dropdown-fix

Conversation

@adescoteaux1

Copy link
Copy Markdown
Contributor

Summary

  • The previous availability-badge PR (Show applicant availability on preference lists, not lead availability #176) only fetched answers for applicants already on the list. The add-applicant pickers show the opposite set (candidates not yet added), so every option in those dropdowns had no data to compute a badge from and never showed one.
  • Include the current tab's whole candidate pool in the same batched answers fetch, so both existing entries and add-picker candidates get a badge from one request.

Test plan

  • npx eslint, npx tsc --noEmit, npx prettier --check, npm run test — all clean

Availability answers were only fetched for applicants already on the
list, so every candidate shown in the "add an applicant" pickers
(precisely the ones NOT yet on the list) never got a badge. Include
the current tab's whole candidate pool in the batch fetch too.
@netlify

netlify Bot commented Aug 22, 2026

Copy link
Copy Markdown

Deploy Preview for generate-appportal ready!

Name Link
🔨 Latest commit a79f0e2
🔍 Latest deploy log https://app.netlify.com/projects/generate-appportal/deploys/6a8a23587646130008ab7577
😎 Deploy Preview https://deploy-preview-177--generate-appportal.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@adescoteaux1
adescoteaux1 merged commit 3a64007 into main Aug 22, 2026
9 checks passed
@adescoteaux1
adescoteaux1 deleted the ally/preference-list-availability-dropdown-fix branch August 22, 2026 22: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.

1 participant