feat: trip persistence — save / list / view saved trips #15

Merged

charles merged 1 commit from feat/trip-persistence into main 2026-06-17 14:32:50 +00:00

Conversation 0 Commits 1 Files changed 8 +312 -9

charles commented 2026-06-17 14:31:09 +00:00

Owner

Copy link

Built trips now persist and are listable per account (FR-PRES-14/17).

Changes

• service: TripStore port (save/get/list, owner-scoped) + TripSummary; mem (MemTrips) + Postgres (trips table, migration 0002, JSONB) impls. Trips capability gains save_trip/list_trips/get_trip (gated). AppContext gains trips_store.
• api: build → save_trip → redirect /trips/:id; GET /trips (list), GET /trips/:id (saved trip, owner-scoped).

Verification

• just qa green. Mem test: owner-scoping (list/get isolation, anon gated). Real Postgres integration: save→get→list, cross-owner denied, status→Saved. Live: build → /trips/1 → list shows "Voyage #1 — 2 étapes · ≈ 500 €".

Traces: FR-PRES-14/17, FR-ACC-09 · tech-spec 02.

Built trips now **persist** and are listable per account (FR-PRES-14/17). ## Changes - **service**: `TripStore` port (save/get/list, **owner-scoped**) + `TripSummary`; `mem` (`MemTrips`) + **Postgres** (trips table, migration 0002, JSONB) impls. `Trips` capability gains `save_trip`/`list_trips`/`get_trip` (gated). `AppContext` gains `trips_store`. - **api**: build → `save_trip` → redirect `/trips/:id`; `GET /trips` (list), `GET /trips/:id` (saved trip, owner-scoped). ## Verification - `just qa` green. Mem test: owner-scoping (list/get isolation, anon gated). **Real Postgres** integration: save→get→list, cross-owner denied, status→Saved. Live: build → `/trips/1` → list shows "Voyage #1 — 2 étapes · ≈ 500 €". Traces: FR-PRES-14/17, FR-ACC-09 · tech-spec 02.

charles added 1 commit 2026-06-17 14:31:09 +00:00

feat(service,api): trip persistence — save / list / view saved trips ... f9aabb50e9

- service: TripStore port (save/get/list, owner-scoped) + TripSummary; mem
  (MemTrips) + Postgres (trips table, migration 0002, JSONB) impls. Trips
  capability gains save_trip/list_trips/get_trip (all gated, owner-scoped).
  AppContext gains trips_store.
- api: build now persists the trip (build_trip → save_trip) and redirects to
  /trips/:id; GET /trips lists the account's trips; GET /trips/:id renders a
  saved trip (owner-scoped). trip_view links back to /trips.

Tests: mem owner-scoping (list/get isolation, anon gated) + real-Postgres
integration (save→get→list, cross-owner denied, status→Saved). Verified live:
build → /trips/1 → list shows "Voyage #1 — 2 étapes · ≈ 500 €". just qa green.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

charles merged commit 5bcc8d1a11 into main 2026-06-17 14:32:50 +00:00

charles referenced this pull request from a commit 2026-06-17 14:32:51 +00:00

feat: trip persistence — save / list / view saved trips (#15)

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels

No items

No labels

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

1 participant

Notifications

Subscribe

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

charles/geneatrip!15

No description provided.