nalalangan/grocery-app
1
0
Fork 0
Code Issues Pull Requests 3 Actions Packages Projects Releases Wiki Activity

Add store item bulk delete mode #9

Merged
nalalangan merged 1 commits from feature/store-item-bulk-delete into feature-custom-store-locations 2026-05-31 17:01:06 -09:00
Conversation 0 Commits 1 Files Changed 5 +237 -60
nalalangan commented 2026-05-31 17:00:25 -09:00
Owner
Copy Link

Owner: Codex
Status: review
Branch: feature/store-item-bulk-delete
Branch relationship: stacked on parent branch feature-custom-store-locations
Likely modified areas: Manage store item catalog modal, store item catalog styles, Playwright catalog modal test, frontend modal docs
Actual modified files: docs/README.md; docs/guides/management-modal-patterns.md; frontend/src/components/manage/StoreAvailableItemsManager.jsx; frontend/src/styles/components/manage/StoreAvailableItemsManager.css; frontend/tests/available-items-catalog.spec.ts
Collision risk: medium
Last meaningful update: 2026-05-31 - replaced per-item edit/delete buttons with row editing plus bulk delete selection and documented the management modal pattern.

Summary:

  • Removes per-item Edit Settings and Delete Item buttons from the store item manager list.
  • Makes each item row open its settings in normal mode.
  • Adds a Delete Items mode above the list with item select/deselect behavior, selected count, Cancel, and slide-confirm deletion.
  • Keeps delete controls admin-scoped and preserves toast feedback.
  • Adds reusable documentation for management modal layout, row behavior, and bulk delete patterns.
  • Updates focused Playwright coverage for row editing and bulk delete mode.

Tests run:

  • npm run lint
  • npm run typecheck
  • npm run build:frontend
  • PLAYWRIGHT_BASE_URL=http://127.0.0.1:3910 npm run test:e2e -- available-items-catalog.spec.ts

Notes:

  • The first focused Playwright run failed because its old fuzzy role assertion matched the new row accessible labels; the assertion was tightened and the rerun passed.
  • Vite build completed with the existing Node version warning: Node.js 20.13.1 is below Vite's recommended 20.19+ or 22.12+.
Owner: Codex Status: review Branch: feature/store-item-bulk-delete Branch relationship: stacked on parent branch feature-custom-store-locations Likely modified areas: Manage store item catalog modal, store item catalog styles, Playwright catalog modal test, frontend modal docs Actual modified files: docs/README.md; docs/guides/management-modal-patterns.md; frontend/src/components/manage/StoreAvailableItemsManager.jsx; frontend/src/styles/components/manage/StoreAvailableItemsManager.css; frontend/tests/available-items-catalog.spec.ts Collision risk: medium Last meaningful update: 2026-05-31 - replaced per-item edit/delete buttons with row editing plus bulk delete selection and documented the management modal pattern. Summary: - Removes per-item Edit Settings and Delete Item buttons from the store item manager list. - Makes each item row open its settings in normal mode. - Adds a Delete Items mode above the list with item select/deselect behavior, selected count, Cancel, and slide-confirm deletion. - Keeps delete controls admin-scoped and preserves toast feedback. - Adds reusable documentation for management modal layout, row behavior, and bulk delete patterns. - Updates focused Playwright coverage for row editing and bulk delete mode. Tests run: - npm run lint - npm run typecheck - npm run build:frontend - PLAYWRIGHT_BASE_URL=http://127.0.0.1:3910 npm run test:e2e -- available-items-catalog.spec.ts Notes: - The first focused Playwright run failed because its old fuzzy role assertion matched the new row accessible labels; the assertion was tightened and the rerun passed. - Vite build completed with the existing Node version warning: Node.js 20.13.1 is below Vite's recommended 20.19+ or 22.12+.
nalalangan added 1 commit 2026-05-31 17:00:25 -09:00
nalalangan merged commit fe9f1eeacc into feature-custom-store-locations 2026-05-31 17:01:06 -09:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: nalalangan/grocery-app#9
No description provided.
Delete Branch "feature/store-item-bulk-delete"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?