Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: adjust when we log warn for gift notification #2638

Merged
merged 3 commits into from
Feb 5, 2025

Conversation

capJavert
Copy link
Contributor

Split condition because warn was firing for each user subscription purchase thus spamming a bit too much

We now:

  • early return when user is not gifted, basic case where user bought subscription for themself
  • warn if user was gifter but was already gifter id is missing

@capJavert
Copy link
Contributor Author

@ilasw for sanity check, logic is the same it was just spamming warns too much.

@capJavert capJavert enabled auto-merge (squash) February 5, 2025 09:10
@capJavert capJavert merged commit c839653 into main Feb 5, 2025
8 checks passed
@capJavert capJavert deleted the fix-warn-log-spam branch February 5, 2025 09:17
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.

2 participants