Skip to content

feat(localizations,shared,ui): Change invite members CTA based on available seats#8612

Open
dstaley wants to merge 2 commits into
feat/per-seat-costsfrom
ds.feat/psc-button-text
Open

feat(localizations,shared,ui): Change invite members CTA based on available seats#8612
dstaley wants to merge 2 commits into
feat/per-seat-costsfrom
ds.feat/psc-button-text

Conversation

@dstaley
Copy link
Copy Markdown
Member

@dstaley dstaley commented May 21, 2026

Description

This PR updates the "Send invitations" button text to "Purchase additional seats" when the organization is on a per-seat cost plan and is attempting to invite more members than their current subscription item entitles them to.

Checklist

  • pnpm test runs as expected.
  • pnpm build runs as expected.
  • (If applicable) JSDoc comments have been added or updated for any package exports
  • (If applicable) Documentation has been updated

Type of change

  • 🐛 Bug fix
  • 🌟 New feature
  • 🔨 Breaking change
  • 📖 Refactoring / dependency upgrade / documentation
  • other:

@vercel
Copy link
Copy Markdown

vercel Bot commented May 21, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
clerk-js-sandbox Ready Ready Preview, Comment May 21, 2026 3:44pm

Request Review

@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented May 21, 2026

⚠️ No Changeset found

Latest commit: b73f589

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

return null;
};

export const getPaidSeatsUnitTier = (unitPrice: BillingPlanUnitPrice | null): BillingPlanUnitPriceTier | null => {
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.

It would be great to add a unit test for this helper

return organization.membersCount + organization.pendingInvitationsCount > seatLimit;
};

export const organizationAndInvitationsExceedsPurchasedSeats = (
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.

this one is pretty simple, but perhaps a happy path unit test?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants