[PM-35253] Add organization ability UseInviteLinks#7489
[PM-35253] Add organization ability UseInviteLinks#7489
Conversation
🤖 Bitwarden Claude Code ReviewOverall Assessment: APPROVE This PR adds the Code Review DetailsNo actionable findings. The implementation follows the established pattern for similar recently-added abilities (
|
|
New Issues (1)Checkmarx found the following issues in this Pull Request
|
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #7489 +/- ##
==========================================
+ Coverage 59.13% 63.62% +4.48%
==========================================
Files 2077 2077
Lines 91848 91881 +33
Branches 8175 8175
==========================================
+ Hits 54315 58455 +4140
+ Misses 35601 31408 -4193
- Partials 1932 2018 +86 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
…ttps://github.com/bitwarden/server into ac/pm-35253/add-useinvitelinks-organization-ability
| [UsePhishingBlocker] BIT NOT NULL CONSTRAINT [DF_Organization_UsePhishingBlocker] DEFAULT (0), | ||
| [UseDisableSmAdsForUsers] BIT NOT NULL CONSTRAINT [DF_Organization_UseDisableSmAdsForUsers] DEFAULT (0), | ||
| [UseMyItems] BIT NOT NULL CONSTRAINT [DF_Organization_UseMyItems] DEFAULT (0), | ||
| [UseInviteLinks] BIT NOT NULL CONSTRAINT [DF_Organization_UseInviteLinks] DEFAULT (0), |
There was a problem hiding this comment.
❓ PR 7438 has /util/Migrator/DbScripts/2026-04-10_02_AddExemptFromBillingAutomation.sql, which adds the ExemptFromBillingAutomation column to this table. If this PR is deployed with the same release (or later) as PR 7438, the columns will be out of order. Do you know if this deployment will happen before 7438? If not, then the UseInviteLinks column should be placed after the ExemptFromBillingAutomation, and the other objects should be adjusted as well.
There was a problem hiding this comment.
@mkincaid-bw well observed! I have fixed the order of the columns by putting UseInviteLinks after ExemptFromBillingAutomation
|
|
||
| WHILE @RowsAffected > 0 | ||
| BEGIN | ||
| UPDATE TOP (@BatchSize) [dbo].[Organization] |
There was a problem hiding this comment.
Note: I tested this against a backup and it took about a minute to run.
|





🎟️ Tracking
https://bitwarden.atlassian.net/browse/PM-35253
📔 Objective
Add the new organization ability
UseInviteLinksacross the server (schema/migrations, domain + licensing, API models, and tests).A database data migration is included to enable
UseInviteLinksfor existing Enterprise organizations.Related Pricing Service changes: https://github.com/bitwarden/billing-pricing/pull/103
Related Clients changes: bitwarden/clients#20227