Skip to content

Bugfix for newly hosted webhook handler.#68

Merged
themarolt merged 1 commit intomainfrom
hotfix/github-webhooks
Oct 4, 2022
Merged

Bugfix for newly hosted webhook handler.#68
themarolt merged 1 commit intomainfrom
hotfix/github-webhooks

Conversation

@themarolt
Copy link
Copy Markdown
Contributor

Changes proposed ✍️

  • Github webhook body was parsed as javascript object not like before when webhook was deployed as a serverless function REST API handler so a few lines had to be changed to make it work with the new api service hosted webhook endpoint

Checklist ✅

  • Label appropriately with type:feature 🚀, type:enhancement ✨, type:bug 🐞, or type:documentation 📜.
  • Tests are passing.
  • New backend functionality has been unit-tested.
  • Environment variables have been updated
  • 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.

@netlify
Copy link
Copy Markdown

netlify Bot commented Oct 4, 2022

Deploy Preview for open-crowd-prod canceled.

Name Link
🔨 Latest commit e483579
🔍 Latest deploy log https://app.netlify.com/sites/open-crowd-prod/deploys/633be4a6c5882e00099b7532

@netlify
Copy link
Copy Markdown

netlify Bot commented Oct 4, 2022

Deploy Preview for open-devfounders canceled.

Name Link
🔨 Latest commit e483579
🔍 Latest deploy log https://app.netlify.com/sites/open-devfounders/deploys/633be4a64ef5300009da80c8

@themarolt themarolt merged commit 2ea0c34 into main Oct 4, 2022
@themarolt themarolt deleted the hotfix/github-webhooks branch October 4, 2022 08:18
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