Skip to content
This repository has been archived by the owner on Jan 17, 2024. It is now read-only.

AB#138096: Set up WebHook in DummyController #283

Merged
merged 9 commits into from
Dec 5, 2023

Conversation

dcl10
Copy link

@dcl10 dcl10 commented Dec 4, 2023

What type of PR is this? (check all applicable)

  • ♻️ Refactor
  • ✨ Feature
  • 🐛 Bug Fix
  • ⚡️ Optimization
  • 📝 Documentation Update

Description

Add a service to DummyController which acts as a web hook for ending Hutch execution outcome submission to configured URL.

Related Tickets & Documents

Screenshots, example outputs/behaviour etc.

Added/updated tests?

  • ✅ Yes
  • ❌ No, and this is why:
    • no endpoint to send the web hook to for testing purposes.
  • ❓ I need help with writing tests

[optional] What gif best describes this PR or how it makes you feel?

alt_text

dcl10 added 9 commits December 1, 2023 12:29
Signed-off-by: Daniel Lea <[email protected]>
Signed-off-by: Daniel Lea <[email protected]>
Signed-off-by: Daniel Lea <[email protected]>
Signed-off-by: Daniel Lea <[email protected]>
Signed-off-by: Daniel Lea <[email protected]>
Signed-off-by: Daniel Lea <[email protected]>
Signed-off-by: Daniel Lea <[email protected]>
Signed-off-by: Daniel Lea <[email protected]>
Copy link
Contributor

github-actions bot commented Dec 4, 2023

ℹ Linked Title:
AB#138096: Set up WebHook in DummyController

This bot comment serves to ensure the title work item gets hyperlinked.

@dcl10 dcl10 merged commit 1e5f0c5 into main Dec 5, 2023
3 checks passed
@dcl10 dcl10 deleted the feature/dummy-controller-webhooks branch December 5, 2023 11:06
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants