Skip to content

[PM-36865] Removes the feature flag from the constants of the server-side code#7849

Merged
JaredScar merged 2 commits into
mainfrom
ac/pm-36865-feature-flag-from-server
Jun 22, 2026
Merged

[PM-36865] Removes the feature flag from the constants of the server-side code#7849
JaredScar merged 2 commits into
mainfrom
ac/pm-36865-feature-flag-from-server

Conversation

@JaredScar

Copy link
Copy Markdown
Contributor

🎟️ Tracking

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

📔 Objective

Removes feature flag from server constants file

@JaredScar JaredScar requested a review from BTreston June 22, 2026 16:53
@JaredScar JaredScar added the ai-review Request a Claude code review label Jun 22, 2026
@github-actions

github-actions Bot commented Jun 22, 2026

Copy link
Copy Markdown
Contributor

🤖 Bitwarden Claude Code Review

Overall Assessment: APPROVE

This PR removes the obsolete PM36859RefactorOrgCollectionsVaultComponent feature flag constant from src/Core/Constants.cs. Verified there are no remaining references to either the constant name or its string key (pm-36859-refactor-org-collections-vault-component) anywhere in the codebase, so the removal is complete and safe. The change is limited to a single line in a single file with no functional impact.

No findings.

BTreston
BTreston previously approved these changes Jun 22, 2026
@codecov

codecov Bot commented Jun 22, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 61.22%. Comparing base (c4887bf) to head (f446123).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #7849   +/-   ##
=======================================
  Coverage   61.22%   61.22%           
=======================================
  Files        2211     2211           
  Lines       97781    97781           
  Branches     8816     8816           
=======================================
+ Hits        59862    59863    +1     
+ Misses      35796    35795    -1     
  Partials     2123     2123           

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

@sonarqubecloud

Copy link
Copy Markdown

@BTreston BTreston self-requested a review June 22, 2026 21:36
@JaredScar JaredScar merged commit cab6f5b into main Jun 22, 2026
47 checks passed
@JaredScar JaredScar deleted the ac/pm-36865-feature-flag-from-server branch June 22, 2026 22:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ai-review Request a Claude code review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants