Skip to content

feat(User): Favorite API Endpoints - #574

Merged
zigzagdev merged 35 commits into
mainfrom
feat/user-favorite
Aug 18, 2026
Merged

feat(User): Favorite API Endpoints#574
zigzagdev merged 35 commits into
mainfrom
feat/user-favorite

Conversation

@zigzagdev

Copy link
Copy Markdown
Owner

No description provided.

feat: add user_favorite migration for favorite heritages join table
feat: implement Infra layer for user favorites repository / お気に入り機能のInfra層実装
feat: implement Application layer for adding favorites / お気に入り追加のApplication層実装
feat: implement Presentation layer for POST /api/v1/favorites / お気に入り追加のPresentation層実装
…infra

feat: implement Infra layer for favorite heritages list API / お気に入り一覧APIのInfra層実装
…application

feat: implement Infra and Application layers for favorite heritages list / お気に入り一覧のInfra・Application層実装
…presentation

feat: implement Presentation layer for GET /api/v1/favorites / お気に入り一覧のPresentation層実装
feat: implement Infra layer for removing favorites / お気に入り解除のInfra層実装
@zigzagdev zigzagdev changed the title feat(User): Favorite Relations API feat(User): Favorite API Endpoints Aug 16, 2026
@codecov

codecov Bot commented Aug 16, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 97.76786% with 10 lines in your changes missing coverage. Please review.
✅ Project coverage is 64.66%. Comparing base (9c6c1d1) to head (cf39eb3).

Files with missing lines Patch % Lines
...ackages/Features/Controller/FavoriteController.php 85.96% 8 Missing ⚠️
...orldHeritageQueryService_getHeritagesByIdsTest.php 97.67% 1 Missing ⚠️
...Packages/Features/Tests/FavoriteControllerTest.php 99.31% 1 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff              @@
##               main     #574      +/-   ##
============================================
+ Coverage     62.90%   64.66%   +1.76%     
- Complexity     1622     1694      +72     
============================================
  Files           136      147      +11     
  Lines          8410     8858     +448     
============================================
+ Hits           5290     5728     +438     
- Misses         3120     3130      +10     
Files with missing lines Coverage Δ
src/app/Models/User.php 100.00% <100.00%> (ø)
...p/Packages/Domains/Favorite/FavoriteRepository.php 100.00% <100.00%> (ø)
.../Domains/Favorite/Tests/FavoriteRepositoryTest.php 100.00% <100.00%> (ø)
...omains/WorldHeritage/WorldHeritageQueryService.php 96.56% <100.00%> (+0.08%) ⬆️
...ndUseCases/UseCase/Favorite/AddFavoriteUseCase.php 100.00% <100.00%> (ø)
...seCases/UseCase/Favorite/RemoveFavoriteUseCase.php 100.00% <100.00%> (ø)
.../Tests/UseCase/GetFavoriteHeritagesUseCaseTest.php 100.00% <100.00%> (ø)
...s/UseCase/Favorite/GetFavoriteHeritagesUseCase.php 100.00% <100.00%> (ø)
...s/Tests/CommandUseCases/AddFavoriteUseCaseTest.php 100.00% <100.00%> (ø)
...ests/CommandUseCases/RemoveFavoriteUseCaseTest.php 100.00% <100.00%> (ø)
... and 4 more
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

…ication

feat: implement Application layer for removing favorites / お気に入り解除のApplication層実装
…entation

feat: implement Presentation layer for DELETE /api/v1/favorites/{world_heritage_id} / お気に入り解除のPresentation層実装
@zigzagdev zigzagdev self-assigned this Aug 16, 2026
@zigzagdev zigzagdev linked an issue Aug 18, 2026 that may be closed by this pull request
4 tasks

@zigzagdev zigzagdev left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok

@zigzagdev
zigzagdev merged commit 905138c into main Aug 18, 2026
27 checks passed
@zigzagdev
zigzagdev deleted the feat/user-favorite branch August 18, 2026 11:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Favorite Heritage API

1 participant