Skip to content

fix(web): Prevent default group card duplication#173

Merged
itswcl merged 1 commit into
mainfrom
claude/fix-default-group-reorder-duplication
May 13, 2026
Merged

fix(web): Prevent default group card duplication#173
itswcl merged 1 commit into
mainfrom
claude/fix-default-group-reorder-duplication

Conversation

@itswcl

@itswcl itswcl commented May 13, 2026

Copy link
Copy Markdown
Owner

Summary

  • Filter display-only default card IDs and loading placeholders before persisting Default group reorder state.
  • Reuse a case-insensitive default-card guard so polluted group payloads normalize correctly on the web.
  • Add API-side filtering for reserved default card symbols in group responses/replacement plus regression coverage.

Test plan

  • npm --prefix /Users/weilee/Desktop/fg-index/apps/web run lint
  • npm --prefix /Users/weilee/Desktop/fg-index/apps/web run build
  • npm --prefix /Users/weilee/Desktop/fg-index/apps/api-server test -- ticker-groups.test.ts
  • npm --prefix /Users/weilee/Desktop/fg-index/apps/api-server run build
  • Start Vite dev server and verify the app renders in headless Chrome

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@itswcl itswcl merged commit 9d4b247 into main May 13, 2026
2 checks passed
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