Skip to content

Conversation

themarolt
Copy link
Contributor

Changes proposed ✍️

  • If we hit rate limit exception we should pause and delay and only set integration to done/error when we are really done
  • max 7 days per integration onboarding limit

Checklist ✅

  • Label appropriately with Feature, Enhancement, or Bug.
  • Tests are passing.
  • New backend functionality has been unit-tested.
  • Environment variables have been updated:
    • Local frontend configuration: frontend/.env.dist.local, frontend/.env.dist.composed.
    • Local backend: backend/.env.dist.local, backend/.env.dist.composed.
    • Configuration docs have been updated.
    • Team members only: update environment variables in override, staging and production env. files and trigger update config script.
  • API documentation has been updated (if necessary) (see docs on API documentation).
  • Quality standards are met.
  • All changes have been tested in a staging site.
  • All changes are working locally running crowd.dev's Docker local environment.

@themarolt themarolt added the Bug Created by Linear-GitHub Sync label Mar 22, 2023
@themarolt themarolt requested a review from epipav March 22, 2023 11:30
@themarolt themarolt merged commit db0b445 into main Mar 23, 2023
@themarolt themarolt deleted the bugfix/long-running-integrations-with-rate-limiting-C-914 branch March 23, 2023 13:16
joanreyero pushed a commit that referenced this pull request Apr 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Created by Linear-GitHub Sync
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants