-
Notifications
You must be signed in to change notification settings - Fork 0
feat: add support for updating records for stripe #30
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
Conversation
NOTE
|
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.
Really good! I have some minor questions.
airbyte-integrations/connectors/source-stripe/source_stripe/streams.py
Outdated
Show resolved
Hide resolved
airbyte-integrations/connectors/source-stripe/source_stripe/streams.py
Outdated
Show resolved
Hide resolved
airbyte-integrations/connectors/source-stripe/source_stripe/streams.py
Outdated
Show resolved
Hide resolved
airbyte-integrations/connectors/source-stripe/source_stripe/streams.py
Outdated
Show resolved
Hide resolved
NOTE
|
airbyte-integrations/connectors/source-stripe/source_stripe/streams.py
Outdated
Show resolved
Hide resolved
airbyte-integrations/connectors/source-stripe/source_stripe/streams.py
Outdated
Show resolved
Hide resolved
airbyte-integrations/connectors/source-stripe/source_stripe/streams.py
Outdated
Show resolved
Hide resolved
airbyte-integrations/connectors/source-stripe/source_stripe/streams.py
Outdated
Show resolved
Hide resolved
NOTE Sources (95)
Destinations (43)
|
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.
The pr looks good. Nice work.
I have some suggestions.
Also, I have a question. If we get an update event for an existing row will the event have the only new value to update the entry in the warehouse?
airbyte-integrations/connectors/source-stripe/source_stripe/streams.py
Outdated
Show resolved
Hide resolved
airbyte-integrations/connectors/source-stripe/source_stripe/streams.py
Outdated
Show resolved
Hide resolved
airbyte-integrations/connectors/source-stripe/source_stripe/streams.py
Outdated
Show resolved
Hide resolved
airbyte-integrations/connectors/source-stripe/source_stripe/streams.py
Outdated
Show resolved
Hide resolved
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.
Thank you!
6eb22d0
to
b252150
Compare
Description
Add support for getting updates for incremental syncs for the following
stripe
resources using the event APIDocs
Ticket