Skip to content

Fix business edit panel fields not labeled or pre-populated#78

Merged
roy-bme merged 1 commit into
claude/blackmass-corporate-website-caGTRfrom
codex/fix-empty-fields-in-edit-business-form
May 14, 2026
Merged

Fix business edit panel fields not labeled or pre-populated#78
roy-bme merged 1 commit into
claude/blackmass-corporate-website-caGTRfrom
codex/fix-empty-fields-in-edit-business-form

Conversation

@roy-bme
Copy link
Copy Markdown
Owner

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

Motivation

  • The map-side business edit panel rendered a stack of unlabeled empty inputs when entering edit mode because the form state was not initialized from the selected business.
  • Users need to see field labels and the current business values when editing so they can understand and update records reliably.

Description

  • Added a buildFormFromBusiness helper that maps a MapBusiness into string-valued form state (including joining arrays into comma-separated strings).
  • Initialize form state from the selected business in handleSelectFromPin and also populate the form when the Edit button is clicked so edit mode opens with existing values.
  • Replaced the unlabeled input stack with a labeled layout that includes Name, Sector, Decision maker, Decision maker title, Phone, Email, LinkedIn, Est. Monthly Volume, ZIMX fit score, Key suppliers, Key customers, Pain points, Zone (select), and Notes (textarea).
  • Preserved existing save flow while ensuring the form fields are bound to the populated form state before saving.

Testing

  • Ran lint with npm run -s lint, which completed successfully (there is an unrelated existing Next.js no-img-element warning reported).
  • No additional automated tests were added or run in this PR.

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 7:46pm

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 7bcf720 into claude/blackmass-corporate-website-caGTR May 14, 2026
2 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