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

Delay discord integration checks with 50% chance #1809

Merged
merged 2 commits into from
Nov 15, 2023

Conversation

garrrikkotua
Copy link
Contributor

@garrrikkotua garrrikkotua commented Nov 2, 2023

Changes proposed ✍️

What

🤖 Generated by Copilot at 51e55c2

This pull request modifies the integration service to improve the handling of Discord integrations. It adds a random delay for triggering new integration checks to avoid Discord API rate limits, and reduces the check interval from 12 hours to 6 hours to sync member data more frequently.

🤖 Generated by Copilot at 51e55c2

Discord members change
Integration service adapts
Delay or check soon

Why

How

🤖 Generated by Copilot at 51e55c2

  • Reduce the frequency of checking for new or updated members in Discord integrations from 12 hours to 6 hours to improve timeliness and accuracy of member data (link)
  • Introduce a random delay for triggering new integration checks for Discord integrations to reduce the rate of hitting Discord API rate limits and prevent errors and failures (link)
  • Add logging messages to indicate whether the integration check is delayed or not for Discord integrations (link)

Checklist ✅

  • Label appropriately with Feature, Improvement, or Bug.
  • Add screenshots to the PR description for relevant FE changes
  • New backend functionality has been unit-tested.
  • API documentation has been updated (if necessary) (see docs on API documentation).
  • Quality standards are met.

@garrrikkotua garrrikkotua merged commit 41649a0 into main Nov 15, 2023
6 checks passed
@garrrikkotua garrrikkotua deleted the improve/desync-discord-checks branch November 15, 2023 13:36
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.

None yet

1 participant