Skip to content

[PM-35263] Admin Portal: Add checkbox for the InviteLinks ability#7578

Merged
r-tome merged 2 commits intomainfrom
ac/pm-35263/admin-portal-add-useinvitelinks-checkbox
May 4, 2026
Merged

[PM-35263] Admin Portal: Add checkbox for the InviteLinks ability#7578
r-tome merged 2 commits intomainfrom
ac/pm-35263/admin-portal-add-useinvitelinks-checkbox

Conversation

@r-tome
Copy link
Copy Markdown
Contributor

@r-tome r-tome commented May 4, 2026

🎟️ Tracking

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

📔 Objective

Add the UseInviteLinks checkbox to the Admin Portal organization edit form, allowing staff to manually enable or disable invite links per organization.

📸 Screenshots

image

@r-tome r-tome marked this pull request as ready for review May 4, 2026 16:10
@r-tome r-tome requested a review from a team as a code owner May 4, 2026 16:10
@r-tome r-tome requested a review from jrmccannon May 4, 2026 16:10
@r-tome r-tome added the ai-review-vnext Request a Claude code review using the vNext workflow label May 4, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 4, 2026

🤖 Bitwarden Claude Code Review

Overall Assessment: APPROVE

Threads the existing UseInviteLinks organization ability through the Admin Portal edit form by adding the model property, plan-helper mapping, controller assignment, form checkbox, and plan-toggle script entry. The change mirrors the established sibling pattern used for UseMyItems, UsePhishingBlocker, and UseAutomaticUserConfirmation, with no logic or security surface added — feature gating is already enforced at the command layer via OrganizationAbility.UseInviteLinks.

Code Review Details

No findings.

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented May 4, 2026

@r-tome r-tome enabled auto-merge (squash) May 4, 2026 16:18
@r-tome r-tome merged commit 7dc339b into main May 4, 2026
42 checks passed
@r-tome r-tome deleted the ac/pm-35263/admin-portal-add-useinvitelinks-checkbox branch May 4, 2026 16:29
@codecov
Copy link
Copy Markdown

codecov Bot commented May 4, 2026

Codecov Report

❌ Patch coverage is 40.00000% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 59.64%. Comparing base (5289a5f) to head (c1192d2).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
...Admin/AdminConsole/Models/OrganizationEditModel.cs 25.00% 3 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #7578      +/-   ##
==========================================
- Coverage   59.64%   59.64%   -0.01%     
==========================================
  Files        2101     2101              
  Lines       92666    92671       +5     
  Branches     8254     8254              
==========================================
+ Hits        55270    55272       +2     
- Misses      35443    35446       +3     
  Partials     1953     1953              

☔ View full report in Codecov by Sentry.
📢 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

ai-review-vnext Request a Claude code review using the vNext workflow

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants