Skip to content

ref(billing): Remove maxAdminGift frontend limits for gifting#109426

Open
mrduncan wants to merge 1 commit intomasterfrom
mrduncan/remove-max-admin-gifts
Open

ref(billing): Remove maxAdminGift frontend limits for gifting#109426
mrduncan wants to merge 1 commit intomasterfrom
mrduncan/remove-max-admin-gifts

Conversation

@mrduncan
Copy link
Member

Gifting limits are now fully enforced in the backend, so remove the frontend maxAdminGift constants and input clamping. The gift category filter now uses freeEventsMultiple to determine giftability.

Gifting limits are now fully enforced in the backend, so remove the
frontend maxAdminGift constants and input clamping. The gift category
filter now uses freeEventsMultiple to determine giftability.

Co-Authored-By: Claude <noreply@anthropic.com>
@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Feb 26, 2026
.filter(category => {
const categoryInfo = getCategoryInfoFromPlural(category);
return categoryInfo?.maxAdminGift && categoryInfo.freeEventsMultiple;
return categoryInfo?.freeEventsMultiple;
Copy link
Member Author

Choose a reason for hiding this comment

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

Confirmed these are always set together so the result here will be the same by only checking freeEventsMultiple

@mrduncan mrduncan marked this pull request as ready for review February 26, 2026 01:56
@mrduncan mrduncan requested a review from a team as a code owner February 26, 2026 01:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant