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

DEVDOCS-5645: [update] add webhook requirements #2114

Merged
merged 9 commits into from
Dec 15, 2023
Merged

Conversation

bc-traciporter
Copy link
Contributor

@bc-traciporter bc-traciporter commented Dec 12, 2023

DEVDOCS-5645

{Ticket number or summary of work}

What changed?

Added webhook requirements to the overview doc.

Anything else?

Add related PRs, salient notes, ticket numbers, etc.

ping {names}

6juara9
6juara9 previously approved these changes Dec 14, 2023
docs/api-docs/webhooks/about-webhooks.mdx Outdated Show resolved Hide resolved
@bc-traciporter
Copy link
Contributor Author

Thank you @6juara9

bc-tayvs
bc-tayvs previously approved these changes Dec 14, 2023
@bc-tayvs
Copy link

@zvuki WDYT

bc-tgomez
bc-tgomez previously approved these changes Dec 14, 2023
Copy link
Collaborator

@bc-tgomez bc-tgomez left a comment

Choose a reason for hiding this comment

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

Changes to keep or leave behind but overall looks great!

docs/api-docs/webhooks/about-webhooks.mdx Outdated Show resolved Hide resolved
docs/api-docs/webhooks/about-webhooks.mdx Outdated Show resolved Hide resolved
@bc-traciporter bc-traciporter dismissed stale reviews from bc-tgomez and bc-tayvs via d1829cb December 14, 2023 16:44
bc-tgomez
bc-tgomez previously approved these changes Dec 14, 2023
Copy link
Collaborator

@bc-tgomez bc-tgomez left a comment

Choose a reason for hiding this comment

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

👍🏽

docs/api-docs/webhooks/about-webhooks.mdx Outdated Show resolved Hide resolved
docs/api-docs/webhooks/about-webhooks.mdx Outdated Show resolved Hide resolved
@bc-traciporter bc-traciporter merged commit 7850ebb into main Dec 15, 2023
2 checks passed
@bc-traciporter bc-traciporter deleted the DEVDOCS-5645 branch December 15, 2023 14:42
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants