-
Notifications
You must be signed in to change notification settings - Fork 16
Conversation
|
||
# Hubspot | ||
|
||
## Implementation Summary |
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.
I really like the way you linked out to the API docs and summarized the available actions in a single place. What are your thoughts on adding something like a Right to Update
column? Each endpoint can have a read
, update
, or delete
so it's important to make the distinction between update and delete.
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.
Is it the case that we use "Update" to satisfy a "Delete" - ie, updating information where it cannot be deleted? If that's the case I think I'd rather change "Right to Delete" to be inclusive of Update endpoints, otherwise just make this a simple read
/update
/delete
breakdown (without promising it satisfies right to access or right to delete).
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.
I like how each SaaS connector has it's own dedicated page with very relevant API links and action summaries. I fixed a few typos with the Stripe endpoint to avoid the back and forth, so all that's left is your thoughts on the "right to update" addition.
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.
This is a good direction for SaaS connector documentation. Each connector integration is its own mini project so the more documentation we have the better!
Updating SaaS connector docs layout and stub in connectors guides
Purpose
Changes
Checklist
Run Unsafe PR Checks
label has been applied, and checks have passed, if this PR touches any external servicesTicket
Fixes #450