Skip to content

feat(game-skin-admin): two-tier nav + unify KV schema to pieces - #29

Merged
joke-lx merged 1 commit into
mainfrom
release/asset-admin-nav
Sep 4, 2026
Merged

feat(game-skin-admin): two-tier nav + unify KV schema to pieces#29
joke-lx merged 1 commit into
mainfrom
release/asset-admin-nav

Conversation

@joke-lx

@joke-lx joke-lx commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

Follow-up to #27 (fb78d9d + 9ac39ab): promote game-skin-admin two-tier nav that was not included in the first PR.

  • Rename chess-skin-admin -> game-skin-admin (git mv, preserving history).
  • game-skin-admin: single component with top-level tabs Games | Emoji.
    • Games -> GameListView (card grid from GAME_SKIN_REGISTRY) -> GameDetail
      (per-game 3 tabs: preview | replace | upload) + deep-link ?game=.
    • Emoji -> EmojiListView stub (Phase 3 emoji common- packs).
  • useSkinAdmin factory: parameterize 12 chess hard-codes + groupId/assetKeys;
    unify KV schema field name to pieces (fr-aligned).

Lint 0 errors; showcase vite build 31.72s green.

Rename chess-skin-admin -> game-skin-admin (git mv, preserving history)
and restructure per user's spec:

- Two tabs at the top: Games | Emoji (Phase 3 emoji list stays as a stub
  EmojiListView until the emoji KV + Lua action pipeline lands).
- Games tab: GameListView (card grid from GAME_SKIN_REGISTRY) -> click
  through to GameDetail (per-game admin with 3 tabs: preview | replace |
  upload).  (?) store/restore scoped per detail entry; popstate sync
  keeps the ?game deep-link aligned.
- Emoji tab: EmojiListView placeholder for common-<game> emoji packs
  (tag-prefix <gameId>-emoji), wired next phase.
- Unified KV schema field name to pieces (was pieces|assets dual field
  + gameId==='chess' branches).  fr/lib/core/game_kit/skin/game_skin_meta.dart
  always serializes/deserializes pieces; gomoku now uses the same field.

Ve lint 0 errors; showcase vite build 31.72s green.
@joke-lx
joke-lx merged commit b2263c0 into main Sep 4, 2026
2 checks passed
@joke-lx
joke-lx deleted the release/asset-admin-nav branch September 4, 2026 12:41
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