Skip to content

frontend: move vuetify-items.js out of api/v3/ - #729

Merged
ajslater merged 1 commit into
developfrom
claude/move-vuetify-items
May 5, 2026
Merged

ajslater merged 1 commit into
developfrom
claude/move-vuetify-items

Conversation

@ajslater

@ajslater ajslater commented May 5, 2026

Copy link
Copy Markdown
Owner

Summary

Pure file move. `vuetify-items.js` is a presentation-layer adapter — it turns API payloads into Vuetify-friendly `{value, title, ...}` shapes for chips and choice lists. Every other file under `api/v3/` is HTTP / WS plumbing; this module was the odd one out.

Moves `frontend/src/api/v3/vuetify-items.js` → `frontend/src/vuetify-items.js` and updates the three callers (metadata-tags.vue, filter-sub-menu.vue, filter-choice-list.vue).

`git mv` preserves rename detection so `git blame` and `git log --follow` survive.

Test plan

  • `bun run lint` — clean
  • `bun run test:ci` — vitest passes
  • Manual: open the metadata dialog (chips render); open a browser filter sub-menu (choice counts work); open a filter choice list (active state + None row + sort all work)

🤖 Generated with Claude Code

The file is a presentation-layer adapter — it turns API payloads into
``{value, title, ...}`` shapes for Vuetify chips and choice lists. Every
other file under ``api/v3/`` is HTTP / WS plumbing, so this one was the
odd module out. Move it up one level to ``src/vuetify-items.js`` and
update the three callers.

Pure file move + import rewrite — no behaviour change. Git tracks the
rename so blame survives.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@ajslater
ajslater merged commit 8853970 into develop May 5, 2026
3 checks passed
@ajslater
ajslater deleted the claude/move-vuetify-items branch May 11, 2026 00:10
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