Skip to content

[PM-5108] fix: Show all owners and admins in admin portal org view - #8099

Open
r-tome wants to merge 3 commits into
mainfrom
fix/PM-5108-admin-portal-no-owner
Open

[PM-5108] fix: Show all owners and admins in admin portal org view#8099
r-tome wants to merge 3 commits into
mainfrom
fix/PM-5108-admin-portal-no-owner

Conversation

@r-tome

@r-tome r-tome commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

🎟️ Tracking

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

📔 Objective

OrganizationViewModel was filtering owners and admins based on a status
derived from Organization.Status, which caused the Administrators section
to show "No results to list" even when owners existed. We were unable to
pinpoint exactly how an organization can reach the state that triggers this,
but it has been observed across multiple production support tickets.

These changes ensure customer support has all organization details available
when investigating issues:

  • Owners and admins are now shown regardless of their OrganizationUserStatusType.
    The view already renders a Status column per row so the user state remains visible.
  • Organization.Status is now displayed on the org page in the admin portal.
    Previously there was no way to see this from the UI, which made it harder
    to diagnose organization setup issues.

📸 Screenshots

image

r-tome added 2 commits July 30, 2026 13:49
OrganizationViewModel was deriving an expected OrganizationUserStatusType
from org.Status and filtering owners/admins to that single status. This
caused confirmed owners to disappear when an org was in Pending state,
which happens in the reseller flow while awaiting InitPendingOrganization.
@r-tome r-tome added the ai-review Request a Claude code review label Jul 30, 2026
@github-actions

github-actions Bot commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

🤖 Bitwarden Claude Code Review

Overall Assessment: APPROVE

Reviewed the removal of the Organization.Status-derived OrganizationUserStatusType filter in OrganizationViewModel, the new Status row in _ViewInformation.cshtml, and the new OrganizationViewModelTests. Traced every consumer of Owners/Admins/OwnersDetails/AdminsDetails: the Administrators table already renders a per-row Status column, and Owners is only read for POST purposes by the reseller CreateOrganization flow in ProvidersController (fresh user input, no view-model construction) — neither OrganizationsController.Edit nor OrganizationEditModel reads it, so the widened result set carries no functional risk on the readonly reseller field. The controller already passes unfiltered GetManyDetailsByOrganizationAsync results, the new Status value is a Razor-encoded enum inside the existing admin-only details list, and the test namespace and BitAutoData usage match the Admin.Test/Models conventions.

Code Review Details

No code findings.

@r-tome r-tome added the t:bugfix Change Type - Bugfix label Jul 30, 2026
@r-tome
r-tome marked this pull request as ready for review July 30, 2026 16:00
@r-tome
r-tome requested a review from a team as a code owner July 30, 2026 16:00
@r-tome
r-tome requested a review from JaredScar July 30, 2026 16:00
@codecov

codecov Bot commented Jul 30, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 80.00000% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 63.64%. Comparing base (fc4c606) to head (3a5b5ef).

Files with missing lines Patch % Lines
...onsole/Views/Organizations/_ViewInformation.cshtml 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #8099      +/-   ##
==========================================
- Coverage   68.12%   63.64%   -4.49%     
==========================================
  Files        2373     2373              
  Lines      102846   102844       -2     
  Branches     9328     9323       -5     
==========================================
- Hits        70068    65459    -4609     
- Misses      30454    35159    +4705     
+ Partials     2324     2226      -98     

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

@r-tome r-tome added needs-qa and removed ai-review Request a Claude code review labels Aug 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-qa t:bugfix Change Type - Bugfix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants