[PM-37593] Add OrganizationUserStatusTypeNew - 🪓 Revoked#7666
[PM-37593] Add OrganizationUserStatusTypeNew - 🪓 Revoked#7666sven-bitwarden wants to merge 3 commits into
Conversation
🤖 Bitwarden Claude Code ReviewOverall Assessment: APPROVE This PR is Phase 1 scaffolding for the initiative to separate Code Review DetailsNo findings. Reviewer feedback from |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #7666 +/- ##
==========================================
+ Coverage 59.98% 64.54% +4.55%
==========================================
Files 2121 2132 +11
Lines 93543 93905 +362
Branches 8301 8341 +40
==========================================
+ Hits 56113 60611 +4498
+ Misses 35450 31219 -4231
- Partials 1980 2075 +95 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
mkincaid-bw
left a comment
There was a problem hiding this comment.
Couple minor changes
JaredScar
left a comment
There was a problem hiding this comment.
Admin Console changes seem fine (only 2 files)
|
@mkincaid-bw updated - thanks for the reminder especially on the views ✌️ |
|



🎟️ Tracking
PM-37593
📔 Objective
This PR implements part of Phase 1 of the initiative to separate Revoked from OrganizationUserStatusType.
The overall initiative will have us creating a new home for OrganizationUserStatusType (sans Revoked), populating it for revokes and checking it first for restores, moving all organization user status type checks to account for revoked now being a separate field, and then cleaning up.