Skip to content

Conversation

themarolt
Copy link
Contributor

Changes proposed ✍️

  • Handle SIGTERM for API service
  • Handle incoming webhook asynchronously

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.

Base automatically changed from enhancement/handle-SIGTERM-gracefully-C-277 to main December 21, 2022 07:13
@themarolt themarolt merged commit edd6b75 into main Dec 21, 2022
@themarolt themarolt deleted the enhancement/handle-SIGTERM-gracefully-api-C-333 branch December 21, 2022 07:13
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.

1 participant