Skip to content

More enterprise friendly wording#318610

Merged
lramos15 merged 2 commits into
mainfrom
lramos15/naval-mongoose
May 27, 2026
Merged

More enterprise friendly wording#318610
lramos15 merged 2 commits into
mainfrom
lramos15/naval-mongoose

Conversation

@lramos15
Copy link
Copy Markdown
Member

Fix #318433

Copilot AI review requested due to automatic review settings May 27, 2026 18:49
@lramos15 lramos15 enabled auto-merge (squash) May 27, 2026 18:49
@lramos15 lramos15 self-assigned this May 27, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates Copilot chat quota dashboard messaging to use more enterprise-friendly wording for Business/Enterprise users when additional usage (overage) is enabled, aligning with the feedback in #318433.

Changes:

  • Add org-focused callout copy (“Your organization has enabled additional usage…”) for Business/Enterprise users when approaching/exceeding premium chat limits with additional usage enabled.
  • Expand unit test coverage to validate the new wording across entitlement types and quota states.
Show a summary per file
File Description
src/vs/workbench/contrib/chat/browser/chatStatus/chatStatusDashboard.ts Updates quota callout copy selection to show org-specific messaging for Business/Enterprise entitlements.
src/vs/workbench/contrib/chat/test/browser/chatStatusDashboard.test.ts Adds assertions for the new org-specific callout strings for Enterprise/Business scenarios.

Copilot's findings

  • Files reviewed: 2/2 changed files
  • Comments generated: 2

entitlement: ChatEntitlement.Business,
}));

assert.strictEqual(getCalloutText(dashboard.element), 'Once the limit is reached, your organization has enabled additional usage to allow for continued use.');
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
assert.strictEqual(getCalloutText(dashboard.element), 'Once the limit is reached, your organization has enabled additional usage to allow for continued use.');
assert.strictEqual(getCalloutText(dashboard.element), 'Your organization has enabled additional usage, so you can continue once your limit is reached.');

entitlement: ChatEntitlement.Enterprise,
}));

assert.strictEqual(getCalloutText(dashboard.element), 'Once the limit is reached, your organization has enabled additional usage to allow for continued use.');
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
assert.strictEqual(getCalloutText(dashboard.element), 'Once the limit is reached, your organization has enabled additional usage to allow for continued use.');
assert.strictEqual(getCalloutText(dashboard.element), 'Your organization has enabled additional usage, so you can continue once your limit is reached.');

entitlement: ChatEntitlement.Enterprise,
}));

assert.strictEqual(getCalloutText(dashboard.element), 'Your organization has enabled additional usage. Usage will continue until limits reset.');
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
assert.strictEqual(getCalloutText(dashboard.element), 'Your organization has enabled additional usage. Usage will continue until limits reset.');
assert.strictEqual(getCalloutText(dashboard.element), 'Your organization has enabled additional usage. You can continue using GitHub Copilot.');

? localize('quotaAdditionalUsageActive', "Additional budget is configured. Usage will continue until limits reset.")
: localize('quotaBudgetActive', "Premium request budget is configured. Usage will continue until limits reset.");
calloutText.textContent = isEnterpriseUser
? localize('quotaAdditionalUsageActiveEnterprise', "Your organization has enabled additional usage. Usage will continue until limits reset.")
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
? localize('quotaAdditionalUsageActiveEnterprise', "Your organization has enabled additional usage. Usage will continue until limits reset.")
? localize('quotaAdditionalUsageActiveEnterprise', "Your organization has enabled additional usage. You can continue using GitHub Copilot.")

calloutText.textContent = isEnterpriseUser
? localize('quotaAdditionalUsageActiveEnterprise', "Your organization has enabled additional usage. Usage will continue until limits reset.")
: isUsageBasedBilling
? localize('quotaAdditionalUsageActive', "Additional budget is configured. Usage will continue until limits reset.")
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"Usage will continue until limits reset." doesn't really make sense to me. I think we want to convey the user can keep using in overage until the limit resets and then they go back to pulling from their credits.

? localize('quotaAdditionalUsageApproaching', "Once the limit is reached, additional budget will be used.")
: localize('quotaBudgetApproaching', "Once the limit is reached, premium request budget will be used.");
calloutText.textContent = isEnterpriseUser
? localize('quotaAdditionalUsageApproachingEnterprise', "Once the limit is reached, your organization has enabled additional usage to allow for continued use.")
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
? localize('quotaAdditionalUsageApproachingEnterprise', "Once the limit is reached, your organization has enabled additional usage to allow for continued use.")
? localize('quotaAdditionalUsageApproachingEnterprise', "Your organization has enabled additional usage, so you can continue once your limit is reached.")

@lramos15 lramos15 merged commit 3345e39 into main May 27, 2026
25 checks passed
@lramos15 lramos15 deleted the lramos15/naval-mongoose branch May 27, 2026 20:26
@vs-code-engineering vs-code-engineering Bot added this to the 1.123.0 milestone May 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Additional budget strings for CB/CE users

3 participants