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

feat: add hubspot webhook config (contains migration) #2035

Merged
merged 4 commits into from
Dec 6, 2023
Merged

feat: add hubspot webhook config (contains migration) #2035

merged 4 commits into from
Dec 6, 2023

Conversation

asdfryan
Copy link
Contributor

@asdfryan asdfryan commented Dec 6, 2023 •
edited

Context

  • We need the developer App Id and developer token for subscribing to Hubspot webhooks
  • We need to store the app Id as a top-level field on the Provider so we can quickly fetch it upon receiving a hubspot webhook (so we can validate the signature, and figure out which application it is)
  • We should also implement a unique constraint on that Provider -- I've done this at the application layer rather the DB layer for now just in case we change our mind
image

@vercel Vercel
Copy link

vercel bot commented Dec 6, 2023 •
edited

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

2 Ignored Deployments
Name Status Preview Comments Updated (UTC)
management-ui ⬜️ Ignored (Inspect) Visit Preview Dec 6, 2023 7:35pm
supaglue-docs ⬜️ Ignored (Inspect) Visit Preview Dec 6, 2023 7:35pm

@asdfryan asdfryan merged commit 345ae76 into main Dec 6, 2023
13 checks passed
@asdfryan asdfryan deleted the hubspot_provider branch December 6, 2023 19:55
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@tonyxiao tonyxiao tonyxiao approved these changes

@tomkit tomkit Awaiting requested review from tomkit tomkit is a code owner

@lucasmarshall lucasmarshall Awaiting requested review from lucasmarshall

@george-xing george-xing Awaiting requested review from george-xing george-xing is a code owner

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

Successfully merging this pull request may close these issues.

None yet
2 participants
@asdfryan @tonyxiao