Skip to content

Twitter rate limit handling#292

Merged
themarolt merged 4 commits intomainfrom
bugfix/twitter-rate-limit-handling-C-188
Dec 1, 2022
Merged

Twitter rate limit handling#292
themarolt merged 4 commits intomainfrom
bugfix/twitter-rate-limit-handling-C-188

Conversation

@themarolt
Copy link
Copy Markdown
Contributor

@themarolt themarolt commented Dec 1, 2022

Changes proposed ✍️

  • Twitter API rate limit handling
  • Small refactor of slack rate limit handling so that we don't repeat the common code
  • fixed formatting in PR GitHub template

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](https://docs.crowd.dev/docs/updating-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 Dec 1, 2022
@themarolt themarolt changed the title Bugfix/twitter rate limit handling c 188 Twitter rate limit handling Dec 1, 2022
@themarolt themarolt requested a review from epipav December 1, 2022 08:42
@themarolt themarolt marked this pull request as ready for review December 1, 2022 08:57
@themarolt themarolt merged commit c51fc62 into main Dec 1, 2022
@themarolt themarolt deleted the bugfix/twitter-rate-limit-handling-C-188 branch December 1, 2022 14:40
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