Skip to content

Explicitly fire automations in github webhooks API route#1108

Merged
garrrikkotua merged 2 commits intomainfrom
fix/github-webhooks
Jul 12, 2023
Merged

Explicitly fire automations in github webhooks API route#1108
garrrikkotua merged 2 commits intomainfrom
fix/github-webhooks

Conversation

@garrrikkotua
Copy link
Copy Markdown
Contributor

@garrrikkotua garrrikkotua commented Jul 12, 2023

Changes proposed ✍️

What

🤖 Generated by Copilot at f908a00

Added support for testing webhook integrations from the UI by passing a flag to the node worker message handler. Modified backend/src/api/webhooks/github.ts to set the flag for test messages.

🤖 Generated by Copilot at f908a00

sendNodeWorkerMessage
Adds a flag for test webhook
Autumn of debugging

Why

How

🤖 Generated by Copilot at f908a00

  • Add a parameter to sendNodeWorkerMessage to indicate test messages (link)
  • Use the parameter to create a NodeWorkerProcessWebhookMessage with a test flag (link)

Checklist ✅

  • Label appropriately with Feature, Improvement, or Bug.
  • Add screehshots 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 3e0044f into main Jul 12, 2023
@garrrikkotua garrrikkotua deleted the fix/github-webhooks branch July 12, 2023 15:45
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