Skip to content

feat(recruiter): support multiple locations in opportunity edit screen#5452

Open
rebelchris wants to merge 2 commits intomainfrom
eng-597-opportunity-edit-screen-where-we-ask-for-location-currently
Open

feat(recruiter): support multiple locations in opportunity edit screen#5452
rebelchris wants to merge 2 commits intomainfrom
eng-597-opportunity-edit-screen-where-we-ask-for-location-currently

Conversation

@rebelchris
Copy link
Contributor

@rebelchris rebelchris commented Feb 4, 2026

Summary

  • Update the opportunity edit form to support adding, editing, and removing multiple locations instead of just one
  • Replace singular location fields with a locations array in the form schema
  • Implement multi-location UI with add/remove buttons using useFieldArray
  • Update data transformation functions to handle locations array with externalLocationId for backend

Test plan

  • Open opportunity edit screen and verify existing locations are loaded correctly
  • Add a new location and verify it appears in the form
  • Remove a location and verify it is removed
  • Save the opportunity and verify all locations are persisted

Closes ENG-597

Created by Huginn 🐦‍⬛

Preview domain

https://eng-597-opportunity-edit-screen.preview.app.daily.dev

rebelchris and others added 2 commits February 4, 2026 13:11
Update the opportunity edit form to support adding, editing, and
removing multiple locations instead of just one.

- Add LocationEntry schema type for location array items
- Replace singular location fields with locations array in schema
- Update opportunityToFormData to map all locations from opportunity
- Update formDataToPreviewOpportunity to transform locations array
- Update formDataToMutationPayload to send locations array with
  externalLocationId to backend
- Implement multi-location UI with useFieldArray in RoleInfoSection
- Add "Add location" button and remove button for each location

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Add required properties to createTestSettings fixture to match
SettingsContextData interface:
- companionExpanded, sortCommentsBy (from RemoteSettings)
- updateSortCommentsBy, updateFlag, updateFlagRemote, updatePromptFlag,
  onToggleHeaderPlacement, setSettings, applyThemeMode (required methods)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@rebelchris rebelchris requested a review from a team as a code owner February 4, 2026 13:35
@vercel
Copy link

vercel bot commented Feb 4, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
daily-webapp Ready Ready Preview Feb 4, 2026 1:38pm
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
storybook Ignored Ignored Feb 4, 2026 1:38pm

Request Review

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