Skip to content

[PM-34809] Remove feature flag for policy drawers from server - #8188

Merged
JaredScar merged 1 commit into
mainfrom
ac/pm-34809-remove-step-2-feature-flag-policy-drawers
Aug 17, 2026
Merged

[PM-34809] Remove feature flag for policy drawers from server#8188
JaredScar merged 1 commit into
mainfrom
ac/pm-34809-remove-step-2-feature-flag-policy-drawers

Conversation

@JaredScar

Copy link
Copy Markdown
Contributor

🎟️ Tracking

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

📔 Objective

Removes feature flag from enum on server-side for policy drawers

@JaredScar JaredScar added ai-review Request a Claude code review t:feature Change Type - Feature Development labels Aug 11, 2026
@github-actions

github-actions Bot commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

🤖 Bitwarden Claude Code Review

Overall Assessment: APPROVE

This PR removes the PolicyDrawers feature flag constant (pm-34804-policy-drawers) from FeatureFlagKeys in src/Core/Constants.cs. A repository-wide search confirms no remaining references to either the constant or the flag key string, so nothing breaks from the removal. No findings.

@codecov

codecov Bot commented Aug 11, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 63.89%. Comparing base (38eaee5) to head (433d303).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #8188   +/-   ##
=======================================
  Coverage   63.89%   63.89%           
=======================================
  Files        2359     2359           
  Lines      102319   102319           
  Branches     9240     9240           
=======================================
  Hits        65378    65378           
  Misses      34714    34714           
  Partials     2227     2227           

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

@JaredScar
JaredScar merged commit 96d213e into main Aug 17, 2026
49 of 53 checks passed
@JaredScar
JaredScar deleted the ac/pm-34809-remove-step-2-feature-flag-policy-drawers branch August 17, 2026 20:41
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 t:feature Change Type - Feature Development

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants