addwin/promotions
1
0
Fork 0
Code Issues Pull requests 1 Projects Releases Packages Wiki Activity Actions

FIX(promotions): default event status promotion_amount to 0 #2

Merged
Ghasemi merged 3 commits from feature/allow-users into master 2026-08-03 11:04:18 -04:00
Conversation 0 Commits 3 Files changed 1 +11 -5

3 commits

Author SHA1 Message Date
Ali Asadi
54de7fdb48 PERF(promotions): collapse allowed-user check into a single query 
Fold the restricted-recipient access check into the recipients
queryset itself via Q(public OR restricted-and-allowed), joining
against AllowedUser instead of running a separate lookup query.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
 2026-08-03 18:21:23 +03:30
Ali Asadi
33aa0ed1b4 PERF(promotions): batch allowed-user lookup in get_configured_promotion_amount 
Checking each recipient via Recipient.is_user_allowed() inside the
loop issued one AllowedUser query per restricted recipient (N+1).
Fetch all matching AllowedUser rows for the plan's recipients in a
single query up front instead.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
 2026-08-03 17:55:42 +03:30
Ali Asadi
ac37f15bea FIX(promotions): default event status promotion_amount to 0 
get_event_status_for_user returned None whenever the event was
already processed, no plan matched, or the user lacked access to the
recipient. Callers now always get a concrete amount: the real payout
if accessible, 0 otherwise.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
 2026-08-03 17:33:59 +03:30