Skip to content

Make Add Business location editable while preserving reverse-geocode autofill#76

Merged
roy-bme merged 1 commit into
claude/blackmass-corporate-website-caGTRfrom
codex/make-location-field-editable-in-add-business-form
May 14, 2026
Merged

Make Add Business location editable while preserving reverse-geocode autofill#76
roy-bme merged 1 commit into
claude/blackmass-corporate-website-caGTRfrom
codex/make-location-field-editable-in-add-business-form

Conversation

@roy-bme
Copy link
Copy Markdown
Owner

@roy-bme roy-bme commented May 14, 2026

Motivation

  • The Add Business dialog previously displayed the resolved location as a read-only block so users could not correct or edit the address before saving.
  • The intent is to allow manual edits while keeping the reverse-geocode value as the default when a pin is dropped.

Description

  • Added a new addressInput state and wired it to an editable Input field so the Location is now a standard text input (type="text").
  • Kept reverse-geocode behavior by writing the resolved address into addressInput when reverseGeocode succeeds and falling back to an empty string when it fails.
  • Updated the submission payload to use addressInput.trim() || null instead of the previous read-only address value and reset addressInput on dialog open/close.
  • Handled nullable geocode results safely (setAddressInput(resolved ?? "")) to satisfy TypeScript and avoid runtime nulls.

Testing

  • Ran the TypeScript compile check with pnpm -s exec tsc --noEmit, which completed successfully.

Codex Task

@vercel
Copy link
Copy Markdown

vercel Bot commented May 14, 2026

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

Project Deployment Actions Updated (UTC)
blackmasswebsite Ready Ready Preview, Comment May 14, 2026 6:17pm

Request Review

@supabase
Copy link
Copy Markdown

supabase Bot commented May 14, 2026

This pull request has been ignored for the connected project ipqmdinidqpmchggjdov because there are no changes detected in supabase directory. You can change this behaviour in Project Integrations Settings ↗︎.


Preview Branches by Supabase.
Learn more about Supabase Branching ↗︎.

@roy-bme roy-bme merged commit 0ec6a8c into claude/blackmass-corporate-website-caGTR May 14, 2026
3 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant