Skip to content

chore: add webhook in payload#17468

Merged
PeerRich merged 7 commits into
mainfrom
chore/add-webhook-in-payload
Nov 4, 2024
Merged

chore: add webhook in payload#17468
PeerRich merged 7 commits into
mainfrom
chore/add-webhook-in-payload

Conversation

@Udit-takkar
Copy link
Copy Markdown
Contributor

@Udit-takkar Udit-takkar commented Nov 4, 2024

What does this PR do?

  • Fixes #XXXX (GitHub issue number)
  • Fixes CAL-XXXX (Linear issue number - should be visible at the bottom of the GitHub issue description)

Mandatory Tasks (DO NOT REMOVE)

  • I have self-reviewed the code (A decent size PR without self-review might be rejected).
  • N/A I have updated the developer docs in /docs if this PR makes changes that would require a documentation change. If N/A, write N/A here and check the checkbox.
  • N/A I confirm automated tests are in place that prove my fix is effective or that my feature works.

How should this be tested?

@dosubot dosubot Bot added the webhooks area: webhooks, callback, webhook payload label Nov 4, 2024
@keithwillcode keithwillcode added core area: core, team members only enterprise area: enterprise, audit log, organisation, SAML, SSO labels Nov 4, 2024
@Udit-takkar Udit-takkar added this to the v4.7 milestone Nov 4, 2024
@vercel
Copy link
Copy Markdown

vercel Bot commented Nov 4, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

2 Skipped Deployments
Name Status Preview Comments Updated (UTC)
cal ⬜️ Ignored (Inspect) Visit Preview Nov 4, 2024 3:48pm
calcom-web-canary ⬜️ Ignored (Inspect) Visit Preview Nov 4, 2024 3:48pm

@graphite-app graphite-app Bot requested a review from a team November 4, 2024 14:46
@graphite-app
Copy link
Copy Markdown

graphite-app Bot commented Nov 4, 2024

Graphite Automations

"Add consumer team as reviewer" took an action on this PR • (11/04/24)

1 reviewer was added to this PR based on Keith Williams's automation.

"Add ready-for-e2e label" took an action on this PR • (11/04/24)

1 label was added to this PR based on Keith Williams's automation.

Comment on lines +76 to +78
webhook: {
...webhook,
secret: undefined,
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

do not expose secret

@Udit-takkar Udit-takkar added the 🚨 urgent Created by Linear-GitHub Sync label Nov 4, 2024
Comment on lines +133 to +134
const dailyVideoReference =
booking.references?.filter((reference) => reference.type === "daily_video")?.pop() ?? null;
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

just fetch the last reference of cal video

PeerRich
PeerRich previously approved these changes Nov 4, 2024
@PeerRich PeerRich enabled auto-merge (squash) November 4, 2024 15:27
@PeerRich PeerRich deleted the chore/add-webhook-in-payload branch November 4, 2024 16:12
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Nov 4, 2024

E2E results are ready!

keithwillcode pushed a commit that referenced this pull request Nov 4, 2024
* chore: add webhook in payload

* chore

* chore: add more fields

* add host email

* chore: change name
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

core area: core, team members only enterprise area: enterprise, audit log, organisation, SAML, SSO ready-for-e2e 🚨 urgent Created by Linear-GitHub Sync webhooks area: webhooks, callback, webhook payload

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants