Skip to content

[PM-33086/7] Remove the feature flag RefactorOrgAcceptInit#7287

Merged
r-tome merged 1 commit intomainfrom
ac/pm-33086/remove-feature-flag
Mar 26, 2026
Merged

[PM-33086/7] Remove the feature flag RefactorOrgAcceptInit#7287
r-tome merged 1 commit intomainfrom
ac/pm-33086/remove-feature-flag

Conversation

@r-tome
Copy link
Copy Markdown
Contributor

@r-tome r-tome commented Mar 24, 2026

🎟️ Tracking

https://bitwarden.atlassian.net/browse/PM-33086
https://bitwarden.atlassian.net/browse/PM-33087

📔 Objective

Remove the RefactorOrgAcceptInit feature flag, keeping the VNext code path and deleting the legacy InitPendingOrganizationAsync method and its associated tests.

@r-tome r-tome marked this pull request as ready for review March 24, 2026 14:26
@r-tome r-tome requested a review from a team as a code owner March 24, 2026 14:26
@r-tome r-tome requested a review from JimmyVo16 March 24, 2026 14:26
@sonarqubecloud
Copy link
Copy Markdown

@github-actions
Copy link
Copy Markdown
Contributor

Logo
Checkmarx One – Scan Summary & Detailsa26d4d82-b1c4-447e-9069-793484949405


Fixed Issues (2) Great job! The following issues were fixed in this Pull Request
Severity Issue Source File / Package
MEDIUM CSRF /src/Api/AdminConsole/Controllers/OrganizationUsersController.cs: 307
MEDIUM CSRF /src/Api/AdminConsole/Controllers/OrganizationUsersController.cs: 307

@codecov
Copy link
Copy Markdown

codecov bot commented Mar 24, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 57.80%. Comparing base (2429b45) to head (4f2a864).
⚠️ Report is 69 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #7287      +/-   ##
==========================================
- Coverage   57.84%   57.80%   -0.05%     
==========================================
  Files        2043     2043              
  Lines       89866    89796      -70     
  Branches     7988     7977      -11     
==========================================
- Hits        51983    51906      -77     
- Misses      36024    36035      +11     
+ Partials     1859     1855       -4     

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

@r-tome r-tome added needs-qa and removed needs-qa labels Mar 25, 2026
@r-tome r-tome merged commit e09e635 into main Mar 26, 2026
46 of 47 checks passed
@r-tome r-tome deleted the ac/pm-33086/remove-feature-flag branch March 26, 2026 12:36
@djsmith85 djsmith85 added the t:feature Change Type - Feature Development label Apr 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

t:feature Change Type - Feature Development

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants