bookmarks: account-synced saved locations API #117

Merged

admin_emi merged 9 commits from feat/location-bookmarks into dev

2026-07-26 18:40:24 +00:00

admin_emi commented

2026-07-26 18:27:48 +00:00

Owner

Summary

Backend half of client-side location bookmarks — lets a signed-in user save locations and have them sync across devices, with a bulk import path to absorb a visitor's pre-existing localStorage bookmarks at login.

New Bookmark ORM model (backend/accounts/models.py), keyed on (user_id, cell_id) like Subscription, plus alembic revision 0004_bookmarks (conditional create, matching the 0003_user_discord_only pattern).
Additive v2 endpoints in backend/accounts/api_accounts.py:
- GET /bookmarks — list the user's saved locations
- POST /bookmarks — create, or upsert-rename if one already exists for that grid cell (200, not 409)
- PATCH /bookmarks/{id} / DELETE /bookmarks/{id} — rename / remove, 404 (never 403) on a bookmark that isn't the caller's
- POST /bookmarks/import — bulk import of a visitor's localStorage bookmarks at login, deduped on cell_id (existing labels are never overwritten), capped at BOOKMARK_MAX_PER_USER
Request/response schemas and limits in backend/accounts/schemas.py.
Route-level tests in backend/tests/accounts/test_bookmarks.py covering CRUD, upsert-on-duplicate, ownership isolation, auth, import dedupe/counts, and the per-user cap.

The frontend half is coming on this same branch.

## Summary Backend half of client-side location bookmarks — lets a signed-in user save locations and have them sync across devices, with a bulk import path to absorb a visitor's pre-existing localStorage bookmarks at login. - New `Bookmark` ORM model (`backend/accounts/models.py`), keyed on `(user_id, cell_id)` like `Subscription`, plus alembic revision `0004_bookmarks` (conditional create, matching the `0003_user_discord_only` pattern). - Additive v2 endpoints in `backend/accounts/api_accounts.py`: - `GET /bookmarks` — list the user's saved locations - `POST /bookmarks` — create, or upsert-rename if one already exists for that grid cell (200, not 409) - `PATCH /bookmarks/{id}` / `DELETE /bookmarks/{id}` — rename / remove, 404 (never 403) on a bookmark that isn't the caller's - `POST /bookmarks/import` — bulk import of a visitor's localStorage bookmarks at login, deduped on `cell_id` (existing labels are never overwritten), capped at `BOOKMARK_MAX_PER_USER` - Request/response schemas and limits in `backend/accounts/schemas.py`. - Route-level tests in `backend/tests/accounts/test_bookmarks.py` covering CRUD, upsert-on-duplicate, ownership isolation, auth, import dedupe/counts, and the per-user cap. The frontend half is coming on this same branch.

admin_emi added 5 commits 2026-07-26 18:27:49 +00:00