Skip to content

[PM-41304] Add feature flag pm-40435-quick-copy-icon-setting - #8121

Merged
jengstrom-bw merged 1 commit into
mainfrom
vault/pm-41304/add-feature-flag
Jul 31, 2026
Merged

[PM-41304] Add feature flag pm-40435-quick-copy-icon-setting#8121
jengstrom-bw merged 1 commit into
mainfrom
vault/pm-41304/add-feature-flag

Conversation

@jengstrom-bw

Copy link
Copy Markdown
Contributor

🎟️ Tracking

Jira

📔 Objective

Add feature flag pm-40435-quick-copy-icon-setting

@jengstrom-bw jengstrom-bw added the t:feature Change Type - Feature Development label Jul 31, 2026
@jengstrom-bw
jengstrom-bw marked this pull request as ready for review July 31, 2026 17:41
@jengstrom-bw
jengstrom-bw merged commit 5f44154 into main Jul 31, 2026
42 of 43 checks passed
@jengstrom-bw
jengstrom-bw deleted the vault/pm-41304/add-feature-flag branch July 31, 2026 17:42
@github-actions

github-actions Bot commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

🤖 Bitwarden Claude Code Review

Overall Assessment: APPROVE

This PR adds a single feature flag constant, PM40435_QuickCopyIconSetting, to the Vault Team section of src/Core/Constants.cs. The constant name and string value follow the established PMXXXXX_Name / pm-xxxxx-name convention used throughout FeatureFlagKeys, and no duplicate key or value exists elsewhere in the codebase. There are no behavioral, security, or data-access changes, and no tests are warranted for a constant-only addition.

Code Review Details

No findings.

@codecov

codecov Bot commented Jul 31, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 62.87%. Comparing base (b4d4c55) to head (e873061).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #8121       +/-   ##
===========================================
+ Coverage   14.99%   62.87%   +47.88%     
===========================================
  Files        1409     2301      +892     
  Lines       61097   100212    +39115     
  Branches     4864     9019     +4155     
===========================================
+ Hits         9161    63008    +53847     
+ Misses      51780    35020    -16760     
- Partials      156     2184     +2028     

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

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

Labels

t:feature Change Type - Feature Development

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants