Skip to content

[PM-30886] - Remove FF for Restoring Users#7071

Merged
jrmccannon merged 4 commits intomainfrom
jmccannon/ac/pm-30886-remove-ff-restore-user-collection
Mar 2, 2026
Merged

[PM-30886] - Remove FF for Restoring Users#7071
jrmccannon merged 4 commits intomainfrom
jmccannon/ac/pm-30886-remove-ff-restore-user-collection

Conversation

@jrmccannon
Copy link
Copy Markdown
Contributor

@jrmccannon jrmccannon commented Feb 24, 2026

🎟️ Tracking

PM-30886

📔 Objective

This removes the feature flagged code around creating default collections when restoring organization users for both bulk and single actions.

@jrmccannon jrmccannon requested a review from a team as a code owner February 24, 2026 20:18
@jrmccannon jrmccannon requested a review from r-tome February 24, 2026 20:18
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Feb 24, 2026

Logo
Checkmarx One – Scan Summary & Details938b6767-9bde-4726-b6ef-27ca79c2b19e


New Issues (1) Checkmarx found the following issues in this Pull Request
# Severity Issue Source File / Package Checkmarx Insight
1 MEDIUM CSRF /src/Api/Auth/Controllers/AccountsController.cs: 452
detailsMethod at line 452 of /src/Api/Auth/Controllers/AccountsController.cs gets a parameter from a user request from model. This parameter value flow...
Attack Vector

@codecov
Copy link
Copy Markdown

codecov Bot commented Feb 24, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 56.45%. Comparing base (0c9b97b) to head (582f5e2).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #7071      +/-   ##
==========================================
- Coverage   56.45%   56.45%   -0.01%     
==========================================
  Files        2013     2013              
  Lines       88112    88109       -3     
  Branches     7863     7862       -1     
==========================================
- Hits        49742    49738       -4     
- Misses      36554    36555       +1     
  Partials     1816     1816              

☔ 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 previously approved these changes Feb 25, 2026
…-collection

# Conflicts:
#	src/Core/AdminConsole/OrganizationFeatures/OrganizationUsers/RestoreUser/v1/RestoreOrganizationUserCommand.cs
#	src/Core/Constants.cs
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Mar 2, 2026

@jrmccannon jrmccannon requested a review from r-tome March 2, 2026 17:30
@jrmccannon jrmccannon merged commit 07ead26 into main Mar 2, 2026
40 checks passed
@jrmccannon jrmccannon deleted the jmccannon/ac/pm-30886-remove-ff-restore-user-collection branch March 2, 2026 17:32
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