-
Notifications
You must be signed in to change notification settings - Fork 17
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
webhooks: add initial documentation #365
Conversation
✅ Deploy Preview for grist-help-preview ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
5debd78
to
6da07ff
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me, I learned a lot about how our webhooks work! 😁
That said, I'm also not familiar with them, so wouldn't be able to tell if anything was missing!
The only thing I'm wondering is if it's worth talking about how reference types are handled? The normal Grist API gives unusual arrays of IDs, rather than resolving and expanding the reference. Not sure if webhooks do the same, or if it's worth talking about.
Overall, really easy to read docs 🙂
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some formatting changes per our Grist Content Style Guide.
ee81a88
to
d1622b4
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great. Thanks!
No description provided.