groombook/web
13
0
Fork 0
Code Issues 1 Pull Requests 1 Actions Packages Projects Releases Wiki Activity

feat(GRO-2159): drag-to-reorder + re-optimize on route planner #63

Merged
Flea Flicker merged 1 commits from feat/GRO-2159-route-drag-reorder into dev 2026-06-09 02:57:50 +00:00
Conversation 0 Commits 1 Files Changed 5
+330 -48
Flea Flicker commented 2026-06-09 02:55:52 +00:00
Member
Copy Link
Copy Source

GRO-2159 — Route Optimization: Drag-to-Reorder + Optimize UX

Adds drag-and-drop reorder and a re-optimize trigger to the /admin/routes planner page (built in GRO-2158).

Changes

  • Deps: @dnd-kit/core, @dnd-kit/sortable, @dnd-kit/utilities
  • Sortable stop list: each stop card has a grab handle (⠿); reorder via drag
  • Sensors: PointerSensor (desktop), TouchSensor (press-and-hold, mobile groomers), KeyboardSensor (a11y) — touch/mobile drag support required by the spec
  • Persist: on drop, PATCH /api/routes/:routeId/reorder with { stopOrder: [routeStopId…] } (full first-to-last order, exact permutation the API requires)
  • Optimistic UI + rollback: list/map update immediately; reverts and shows an error if the PATCH fails. Server response (recomputed legs, buffers, totals, conflict flags) is adopted authoritatively on success
  • Tight-schedule warnings: retained — server flags stops where the gap < travel + buffer
  • Re-optimize: after a manual reorder, a hint banner with a Re-optimize button appears and re-runs POST /api/routes/optimize
  • Tests: drag handles render, conflict flag shows, no pre-reorder hint, reorder endpoint mocked (179 tests pass)

UAT Playbook

Updated UAT_PLAYBOOK.md §5.29 — Route Planner: Drag-to-Reorder & Re-optimize (GRO-2159) (7 new test cases: handle present, reorder persists, optimistic rollback, conflict re-eval, re-optimize button, touch drag, groomer own-route).

Verification

  • pnpm typecheck ✅
  • pnpm lint ✅ (only pre-existing warnings, none in changed files)
  • pnpm test ✅ 179 passed
  • pnpm build ✅

Closes GRO-2159.

## GRO-2159 — Route Optimization: Drag-to-Reorder + Optimize UX Adds drag-and-drop reorder and a re-optimize trigger to the `/admin/routes` planner page (built in GRO-2158). ### Changes - **Deps:** `@dnd-kit/core`, `@dnd-kit/sortable`, `@dnd-kit/utilities` - **Sortable stop list:** each stop card has a grab handle (⠿); reorder via drag - **Sensors:** `PointerSensor` (desktop), `TouchSensor` (press-and-hold, mobile groomers), `KeyboardSensor` (a11y) — touch/mobile drag support required by the spec - **Persist:** on drop, `PATCH /api/routes/:routeId/reorder` with `{ stopOrder: [routeStopId…] }` (full first-to-last order, exact permutation the API requires) - **Optimistic UI + rollback:** list/map update immediately; reverts and shows an error if the PATCH fails. Server response (recomputed legs, buffers, totals, conflict flags) is adopted authoritatively on success - **Tight-schedule warnings:** retained — server flags stops where the gap < travel + buffer - **Re-optimize:** after a manual reorder, a hint banner with a **Re-optimize** button appears and re-runs `POST /api/routes/optimize` - **Tests:** drag handles render, conflict flag shows, no pre-reorder hint, reorder endpoint mocked (179 tests pass) ### UAT Playbook Updated `UAT_PLAYBOOK.md` **§5.29 — Route Planner: Drag-to-Reorder & Re-optimize (GRO-2159)** (7 new test cases: handle present, reorder persists, optimistic rollback, conflict re-eval, re-optimize button, touch drag, groomer own-route). ### Verification - `pnpm typecheck` ✅ - `pnpm lint` ✅ (only pre-existing warnings, none in changed files) - `pnpm test` ✅ 179 passed - `pnpm build` ✅ Closes [GRO-2159](/GRO/issues/GRO-2159).
Flea Flicker added 1 commit 2026-06-09 02:55:53 +00:00
feat(GRO-2159): drag-to-reorder + re-optimize on route planner
CI / Test (pull_request) Successful in 24s
Details
CI / Lint & Typecheck (pull_request) Successful in 31s
Details
CI / Build & Push Docker Image (pull_request) Successful in 50s
Details
2e5e35dcea 
Add @dnd-kit drag-and-drop reorder to the /admin/routes stop list.

- Install @dnd-kit/core, @dnd-kit/sortable, @dnd-kit/utilities
- Sortable stop cards with a grab handle; pointer + touch (press-and-hold)
  + keyboard sensors so mobile groomers and a11y users can reorder
- On drop, PATCH /api/routes/:routeId/reorder { stopOrder } (full order)
- Optimistic UI update with rollback + error message on failure
- Server recomputes legs/buffers/conflicts; response adopted authoritatively
- Tight-schedule conflict warnings retained (server-flagged, gap < travel+buffer)
- "Re-optimize" hint+button after a manual reorder (re-runs POST /optimize)
- Tests: drag handles, conflict flag, no pre-reorder hint, reorder mock
- Updated UAT_PLAYBOOK.md §5.29 — drag-to-reorder & re-optimize cases

Co-Authored-By: Paperclip <noreply@paperclip.ing>
Flea Flicker merged commit 59a29a2d03 into dev 2026-06-09 02:57:50 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
bug
Something isn't working
 documentation
Improvements or additions to documentation
 duplicate
This issue or pull request already exists
 enhancement
New feature or request
 good first issue
Good for newcomers
 help wanted
Extra attention is needed
 invalid
This doesn't seem right
 question
Further information is requested
 wontfix
This will not be worked on
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
ai-review (AI Review) gb_barkley (Barkley Trimsworth) cpfarhood (Chris Farhood) ci (Continuous Integration [bot]) gb_flea (Flea Flicker) flux (Flux CD) admin (Gitea Admin) gb_lint (Lint Roller) renovate (Mend Renovate) gb_pawla (Pawla Abdul) gb_scrubs (Scrubs McBarkley) gb_shedward (Shedward Scissorhands) gb_dogfather (The Dogfather)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: groombook/web#63
Delete Branch "feat/GRO-2159-route-drag-reorder"

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?