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.

Sign up for GitHub

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

Jump to bottom

fix(extensions): support multiple webhooks #345

Merged
merged 2 commits into from Dec 19, 2022
Merged

fix(extensions): support multiple webhooks #345

merged 2 commits into from Dec 19, 2022

Conversation

aleksey-a-maltsev
Copy link
Contributor

@aleksey-a-maltsev aleksey-a-maltsev commented Dec 15, 2022 •
edited

Closes #344

daniloab
daniloab approved these changes Dec 15, 2022
View reviewed changes
Copy link
Collaborator

@daniloab daniloab left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks

@aleksey-a-maltsev
Copy link
Contributor Author

Hi @daniloab! There is also workflow waiting for approval. Thank you!

@daniloab
Copy link
Collaborator

Hi @daniloab! There is also workflow waiting for approval. Thank you!

Done. Can you review, please?

@aleksey-a-maltsev
Copy link
Contributor Author

Hi @daniloab! I fixed linter issue, please run tests again.

@daniloab daniloab merged commit b6e3c54 into Surnet:master Dec 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@daniloab daniloab daniloab approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

x-webhooks generates only one entry in resulting JSON
2 participants
@aleksey-a-maltsev @daniloab