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
✨ Source Shopify: add new stream Disputes #28770
Conversation
…schemas/disputes.json Co-authored-by: sh4sh <6833405+sh4sh@users.noreply.github.com>
Before Merging a Connector Pull RequestWow! What a great pull request you have here! 🎉 To merge this PR, ensure the following has been done/considered for each connector added or updated:
If the checklist is complete, but the CI check is failing,
|
@sh4sh Hello! I closed previous PR, and created this instead. |
# Conflicts: # airbyte-integrations/connectors/source-shopify/Dockerfile # airbyte-integrations/connectors/source-shopify/metadata.yaml
Hey @efimmatytsin thanks for testing the pre-release image. Looks good! I don't have write access to your fork, so I added the required change for acceptance tests in #28803 where they're now running. Once those are passing, we can add the change to this PR and merge from here. |
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.
Need to add acceptance tests bypass i.e. 19cc2da then LGTM 👍
@sh4sh Hello! done! |
@sh4sh Hello! what next? |
Hey @efimmatytsin one last thing - could you please add a changelog entry to the connector doc? https://github.com/airbytehq/airbyte/blob/master/docs/integrations/sources/shopify.md |
@sh4sh Hello! Done. |
@sh4sh Hello, if you need something else, please let me know! |
/approve-and-merge reason="source-shopify failing on master, change tested separately" |
What
We want sync disputes from shopify https://shopify.dev/docs/api/admin-rest/2022-10/resources/dispute#get-shopify-payments-disputes?initiated-at=2013-05-03
How
Create new incremental stream Disputes.
🚨 User Impact 🚨
None.