[PM-40255] Update collection icon to bwi-shared-folder#21996
[PM-40255] Update collection icon to bwi-shared-folder#21996nick-livefront wants to merge 20 commits into
bwi-shared-folder#21996Conversation
🤖 Bitwarden Claude Code ReviewOverall Assessment: APPROVE Reviewed this feature-flagged UI refinement that swaps collection icons to Code Review DetailsNo new findings. The change is mechanical and consistently gated behind the feature flag; the mapping is applied exactly once at each call site, and the Prior threads resolved:
|
Changes in this PR impact the Autofill experience of the browser clientBIT has tested the core experience with these changes and the feature flag configuration used by ✅ Fortunately, these BIT tests have passed! 🎉 |
Changes in this PR impact the Autofill experience of the browser clientBIT has tested the core experience with these changes and all feature flags disabled. ✅ Fortunately, these BIT tests have passed! 🎉 |
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## main #21996 +/- ##
==========================================
- Coverage 52.44% 52.28% -0.16%
==========================================
Files 4130 4158 +28
Lines 130572 131564 +992
Branches 20494 20666 +172
==========================================
+ Hits 68475 68793 +318
- Misses 57045 57707 +662
- Partials 5052 5064 +12 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
jprusik
left a comment
There was a problem hiding this comment.
Approved for Autofill concerns
🎟️ Tracking
PM-40255
📔 Objective
Updates usage of
bwi-collectionandbwi-collection-sharedto bebwi-shared-folderglobally.<i>to<bit-icon>NOTE: There are still hardcoded examples in storybook. These are accounted for in PM-40765 which will be handled post launch work
📸 Screenshots
Not every instance was screenshot.
@bitwarden/team-auth-dev - I wasn't getting any organization ciphers in the emergency view to return from the endpoint so I don't have a screenshot for you. Is that expected? Possibly a unique scenario?
Vault
Admin Console
Autofill
DIRT
Tools