Skip to content

[PM-12473] Add delta-shaped PATCH endpoint for collection access - #8212

Draft
r-tome wants to merge 4 commits into
ac/pm-12473/collection-user-authorization-servicefrom
ac/pm-12473/collection-user-access-delta-endpoint
Draft

[PM-12473] Add delta-shaped PATCH endpoint for collection access#8212
r-tome wants to merge 4 commits into
ac/pm-12473/collection-user-authorization-servicefrom
ac/pm-12473/collection-user-access-delta-endpoint

Conversation

@r-tome

@r-tome r-tome commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

🎟️ Tracking

📔 Objective

📸 Screenshots

@codecov

codecov Bot commented Aug 14, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 95.33074% with 24 lines in your changes missing coverage. Please review.
✅ Project coverage is 68.68%. Comparing base (6392403) to head (76778b3).

Files with missing lines Patch % Lines
.../AdminConsole/Repositories/CollectionRepository.cs 89.74% 8 Missing ⚠️
.../AdminConsole/Repositories/CollectionRepository.cs 94.59% 8 Missing ⚠️
.../AdminConsole/Controllers/CollectionsController.cs 89.09% 1 Missing and 5 partials ⚠️
...roupAccess/ModifyCollectionGroupAccessValidator.cs 98.41% 0 Missing and 1 partial ⚠️
...yUserAccess/ModifyCollectionUserAccessValidator.cs 98.55% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@                                  Coverage Diff                                  @@
##           ac/pm-12473/collection-user-authorization-service    #8212      +/-   ##
=====================================================================================
+ Coverage                                              63.25%   68.68%   +5.43%     
=====================================================================================
  Files                                                   2383     2394      +11     
  Lines                                                 103845   104358     +513     
  Branches                                                9409     9460      +51     
=====================================================================================
+ Hits                                                   65684    71681    +5997     
+ Misses                                                 35928    30339    -5589     
- Partials                                                2233     2338     +105     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 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.

@r-tome
r-tome force-pushed the ac/pm-12473/collection-user-access-delta-endpoint branch from bebf875 to aaf8357 Compare August 14, 2026 10:29
@r-tome
r-tome force-pushed the ac/pm-12473/collection-user-authorization-service branch from 56c0576 to 896ca86 Compare August 14, 2026 10:29
r-tome added 3 commits August 14, 2026 11:48
Adds a new, additive PATCH organizations/{orgId}/collections/{id} endpoint
behind a feature flag, accepting add/update/remove delta-shaped changes to a
collection's user and group access instead of the existing PUT endpoint's
full-list-replace model. Authorizes via ICollectionAuthorizationService.
Backed by new ModifyCollectionUserAccessCommand/ModifyCollectionGroupAccessCommand
(+ validators) and repository support (ModifyUserAccessAsync/ModifyGroupAccessAsync,
new stored procedures/migrations). The existing PUT endpoint is untouched.
… ac/pm-12473/collection-user-access-delta-endpoint
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

t:tech-debt Change Type - Tech debt

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant