Skip to content

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

Open
r-tome wants to merge 1 commit intomainfrom
ac/pm-33086/remove-feature-flag
Open

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

Conversation

@r-tome
Copy link
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

@github-actions
Copy link
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

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants