Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(1769): [2] Add _enqueueWebhook method for queue-service cooperation #99

Merged

Conversation

yk634
Copy link
Contributor

@yk634 yk634 commented Jan 6, 2022

Context

This is the implementation that follows screwdriver-cd/screwdriver#2622.
Implemented a method to pass the webhookConfig from SCM to the queue service endpoint /v1/queue/message?type=webhook.

Objective

Added _enqueueWebhook method called by the SD API.

References

The test fail until the enqueueWebhook method is added to the super class.
Please do not merge this PR unti the leading PR have been merged.

Related PR:

screwdriver-cd/screwdriver#1769 (comment)

License

I confirm that this contribution is made under the terms of the license found in the root directory of this repository's source tree and that I have the authority necessary to make this contribution on behalf of its copyright owner.

@yk634 yk634 changed the title [WIP] feat(1769): Add enqueueWebhook method feat(1769): [2] Add _enqueueWebhook method for queue-service cooperation Jan 7, 2022
@jithine jithine merged commit f780477 into screwdriver-cd:master Jan 11, 2022
@sd-buildbot
Copy link

🎉 This PR is included in version 3.2.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@wahapo wahapo deleted the use-queue-webhook branch January 12, 2022 01:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants