Skip to content

[PM-33147 ] - add feature flag for welcome dialog no ext prompt#7144

Merged
jaasen-livefront merged 1 commit intomainfrom
PM-33147
Mar 4, 2026
Merged

[PM-33147 ] - add feature flag for welcome dialog no ext prompt#7144
jaasen-livefront merged 1 commit intomainfrom
PM-33147

Conversation

@jaasen-livefront
Copy link
Collaborator

🎟️ Tracking

https://bitwarden.atlassian.net/browse/PM-33147

📔 Objective

📸 Screenshots

@jaasen-livefront jaasen-livefront enabled auto-merge (squash) March 4, 2026 19:47
@sonarqubecloud
Copy link

sonarqubecloud bot commented Mar 4, 2026

@github-actions
Copy link
Contributor

github-actions bot commented Mar 4, 2026

Logo
Checkmarx One – Scan Summary & Detailsc9770164-c1b2-47b9-8657-70f61ff03a00


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

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

@jaasen-livefront jaasen-livefront merged commit 2b663d7 into main Mar 4, 2026
43 checks passed
@jaasen-livefront jaasen-livefront deleted the PM-33147 branch March 4, 2026 20:04
@codecov
Copy link

codecov bot commented Mar 4, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 56.68%. Comparing base (44e993e) to head (f86c525).
⚠️ Report is 33 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #7144      +/-   ##
==========================================
- Coverage   56.76%   56.68%   -0.08%     
==========================================
  Files        2014     2025      +11     
  Lines       88218    88640     +422     
  Branches     7855     7905      +50     
==========================================
+ Hits        50073    50242     +169     
- Misses      36321    36572     +251     
- Partials     1824     1826       +2     

☔ 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

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants