Skip to content

feat: session CRUD (DELETE + wishlist remove) + 8 new frame catalog SKUs - #131

Open
laurentketterle-hub wants to merge 1 commit into
mergeos-bounties:masterfrom
laurentketterle-hub:feat/sessions-crud-plus-8-frames
Open

feat: session CRUD (DELETE + wishlist remove) + 8 new frame catalog SKUs#131
laurentketterle-hub wants to merge 1 commit into
mergeos-bounties:masterfrom
laurentketterle-hub:feat/sessions-crud-plus-8-frames

Conversation

@laurentketterle-hub

Copy link
Copy Markdown

Changes

Session API enhancements (Fixes #10 - 100 MRG)

  • DELETE /api/sessions/{id} - delete a session by ID
  • DELETE /api/sessions/{id}/wishlist - remove a frame from wishlist
  • remove_wishlist() / delete_session() / prune_expired() in sessions.py
  • WishlistRemove Pydantic model
  • 23 comprehensive tests: CRUD, wishlist lifecycle, edge cases, timestamps

Catalog expansion (Fixes #3 - 50 MRG)

  • 8 new frame SKUs (cycle 17a-h): Pantos Crystal, Oversized Square Tortoise, Cat-Eye Tortoise Shell, Wayfarer Crystal Blush, Round Sunset Gradient, Aviator Silver Mirror, Rectangle Two-Tone Blue, Oval Pearl White
  • Full metadata: fit, studio_fit, ar_fit, GLB refs, descriptions
  • 4 verification tests

Test results

61 passed in 5.17s - zero regressions

CI

Push-triggered via git push (not Contents API).

- Add DELETE /api/sessions/{id} endpoint
- Add DELETE /api/sessions/{id}/wishlist endpoint
- Add remove_wishlist(), delete_session(), prune_expired()
- Add 23 comprehensive session tests
- Add 8 new catalog frame SKUs (cycle 17a-h)
- Add 4 catalog verification tests

Closes mergeos-bounties#10, Closes mergeos-bounties#3
@laurentketterle-hub

Copy link
Copy Markdown
Author

/attempt 10
/attempt 3

@laurentketterle-hub

Copy link
Copy Markdown
Author

Friendly ping - PR open for 3 days. Anything needed to move this forward? Thanks!

@laurentketterle-hub

Copy link
Copy Markdown
Author

Hi maintainer! Just a friendly ping on this PR - it has been ready for review. Any feedback or updates needed? Thanks!

5 similar comments
@laurentketterle-hub

Copy link
Copy Markdown
Author

Hi maintainer! Just a friendly ping on this PR - it has been ready for review. Any feedback or updates needed? Thanks!

@laurentketterle-hub

Copy link
Copy Markdown
Author

Hi maintainer! Just a friendly ping on this PR - it has been ready for review. Any feedback or updates needed? Thanks!

@laurentketterle-hub

Copy link
Copy Markdown
Author

Hi maintainer! Just a friendly ping on this PR - it has been ready for review. Any feedback or updates needed? Thanks!

@laurentketterle-hub

Copy link
Copy Markdown
Author

Hi maintainer! Just a friendly ping on this PR - it has been ready for review. Any feedback or updates needed? Thanks!

@laurentketterle-hub

Copy link
Copy Markdown
Author

Hi maintainer! Just a friendly ping on this PR - it has been ready for review. Any feedback or updates needed? Thanks!

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.

[100 MRG] Server: wishlist + session API [50 MRG] Catalog: 8 more frames (JSON + SVG)

1 participant