Skip to content

fix(showcase): seed Accounts + task map coordinates; bump objectui pin#1466

Merged
xuyushun441-sys merged 2 commits into
mainfrom
fix/showcase-seed-accounts-and-task-locations
Jun 1, 2026
Merged

fix(showcase): seed Accounts + task map coordinates; bump objectui pin#1466
xuyushun441-sys merged 2 commits into
mainfrom
fix/showcase-seed-accounts-and-task-locations

Conversation

@xuyushun441-sys
Copy link
Copy Markdown
Contributor

Summary

Resolves the two showcase seed-data gaps found while sweeping the example app, plus a related renderer bump.

  1. Accounts list was emptyAccount.status is required: true, but the seed records omitted it (no default is applied at seed-insert time), so all 3 records were rejected. Now sets status explicitly (+ adds hq coordinates).
  2. Map view excluded every task ("10 records with missing or invalid coordinates") — tasks had no location. Each task now carries real { lat, lng }, so the Map renders 10 markers and the warning is gone.
  3. Bump .objectui-sha3442c55 to pick up fix(fields): render location/geolocation cells as coordinates, not [Object] objectui#1435, so the hq/location cells render as "47.6062, -122.3321" (pin icon) instead of [Object].

Verification

Rebuilt the console at the new pin and restarted pnpm dev:

  • Accounts list shows Northwind / Contoso / Fabrikam with industry, revenue, website, and HQ coordinates.
  • Task Map view renders 10 markers, no "excluded" banner.

🤖 Generated with Claude Code

os-zhuang and others added 2 commits June 1, 2026 20:45
- data: Account records omitted the required `status` (no default is applied
  at seed-insert time), so all 3 accounts were rejected and the Accounts list
  was empty. Set `status` explicitly and add `hq` coordinates.
- data: tasks had no `location`, so the Map view excluded every record
  ("missing or invalid coordinates"). Give each task real lat/lng so the map
  renders 10 markers.
- Bump .objectui-sha to pick up objectstack-ai/objectui#1435 (location/
  geolocation cells render as coordinates instead of "[Object]").

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@vercel
Copy link
Copy Markdown

vercel Bot commented Jun 1, 2026

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

Project Deployment Actions Updated (UTC)
spec Ready Ready Preview, Comment Jun 1, 2026 12:53pm

Request Review

@github-actions github-actions Bot added documentation Improvements or additions to documentation tooling labels Jun 1, 2026
@xuyushun441-sys xuyushun441-sys merged commit d7f86db into main Jun 1, 2026
7 of 8 checks passed
@github-actions github-actions Bot added the size/s label Jun 1, 2026
@xuyushun441-sys xuyushun441-sys deleted the fix/showcase-seed-accounts-and-task-locations branch June 1, 2026 12:51
xuyushun441-sys pushed a commit that referenced this pull request Jun 2, 2026
Merge pull request #1466 from objectstack-ai/feat/adr-0033-phase-b-draft-review

objectui@3a8fdc36093c118d7b0bc901eb55f0017d86f046
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation size/s tooling

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants