test title #51

Closed
eliott wants to merge 0 commits from develop into main
Owner

test body

test body
- AsNoTracking on every read-only repo query; new FindAccountStateAsync slim projection
- AccountActiveRequirementFilter shares the resolved state with PermissionRequirementFilter via HttpContext.Items so each authenticated request hits the cache once instead of twice
- Migrate every feature service from RemoveByPatternAsync (Redis SCAN) to tag-based InvalidateTagAsync; new <Feature>CacheKeys per module
- ToggleFavorites: direct join-row insert/delete instead of loading the full favourites graph
- ApiResponse.Ok(...) standardization across controllers; remove the `new ApiResponse<T>(200, ...)` form
- PermissionLabels constants replace stringly-typed permission labels (renamed from Permissions to avoid the cesizen_api.Api.Permissions namespace collision)
- CreateUser: parallelize the independent GetDefaultRole / GetTokenType awaits
- UserRepository: single shared ToReturnDto projection across all four read paths so ListUsers can't drift from the detail endpoints
- RolePerUserService.UpdateUserRoles: compute add/remove deltas instead of full delete-then-insert
- GlobalExceptionHandler: downgrade expected exceptions (KeyNotFound, Auth, Validation, BadHttpRequest) from LogError to LogInformation
- RespirationExerciseService: drop the redundant user cache+DB roundtrip in Update/Delete

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
eliott closed this pull request 2026-05-15 11:45:16 +02:00
Commenting is not possible because the repository is archived.
No reviewers
No labels
No milestone
No project
No assignees
1 participant
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
bloc-cesi/cesizen-api!51
No description provided.