Skip to content

[PM-29653] Align PoliciesController.Put With VNext#7130

Merged
sven-bitwarden merged 1 commit intomainfrom
ac/pm-29653/align-put-policies-vnext
Mar 4, 2026
Merged

[PM-29653] Align PoliciesController.Put With VNext#7130
sven-bitwarden merged 1 commit intomainfrom
ac/pm-29653/align-put-policies-vnext

Conversation

@sven-bitwarden
Copy link
Contributor

@sven-bitwarden sven-bitwarden commented Mar 3, 2026

🎟️ Tracking

PM-29653

📔 Objective

We are gradually phasing out the old PUT policy behavior. Our last move in this regard was to redirect all clients calls to use the VNext API endpoint, now we're executing similar behavior server-side by forward calls to the old API endpoint.

@sven-bitwarden sven-bitwarden requested a review from a team as a code owner March 3, 2026 18:11
@sven-bitwarden sven-bitwarden requested a review from JimmyVo16 March 3, 2026 18:11
@sonarqubecloud
Copy link

sonarqubecloud bot commented Mar 3, 2026

@github-actions
Copy link
Contributor

github-actions bot commented Mar 3, 2026

Logo
Checkmarx One – Scan Summary & Details6f3d1bcf-d578-4b3c-ab1b-ced8391e2ddf

Great job! No new security vulnerabilities introduced in this pull request

@codecov
Copy link

codecov bot commented Mar 3, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 56.57%. Comparing base (c97ccc1) to head (7997a4a).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #7130      +/-   ##
==========================================
- Coverage   60.65%   56.57%   -4.08%     
==========================================
  Files        2016     2016              
  Lines       88416    88411       -5     
  Branches     7882     7881       -1     
==========================================
- Hits        53625    50020    -3605     
- Misses      32887    36572    +3685     
+ Partials     1904     1819      -85     

☔ View full report in Codecov by Sentry.
📢 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.

@codecov
Copy link

codecov bot commented Mar 3, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 56.57%. Comparing base (c97ccc1) to head (7997a4a).
⚠️ Report is 15 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #7130      +/-   ##
==========================================
- Coverage   60.65%   56.57%   -4.08%     
==========================================
  Files        2016     2016              
  Lines       88416    88411       -5     
  Branches     7882     7881       -1     
==========================================
- Hits        53625    50020    -3605     
- Misses      32887    36572    +3685     
+ Partials     1904     1819      -85     

☔ View full report in Codecov by Sentry.
📢 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.

@sven-bitwarden sven-bitwarden merged commit 996f479 into main Mar 4, 2026
47 of 48 checks passed
@sven-bitwarden sven-bitwarden deleted the ac/pm-29653/align-put-policies-vnext branch March 4, 2026 22:54
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.

2 participants