-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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 Amazon Seller Partner: delete deprecated streams #34283
🐛Source Amazon Seller Partner: delete deprecated streams #34283
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Apply Sweep Rules to your PR?This is an automated message generated by Sweep AI. |
Fixing PR: track the progress here.I'm currently fixing this PR to address the following: [Sweep GHA Fix] The GitHub Actions run failed with the following error logs:
An error has occurred: Cmd('git') failed due to: exit code(128) |
Fixing PR: track the progress here.I'm currently fixing this PR to address the following: [Sweep GHA Fix] The GitHub Actions run failed with the following error logs:
An error has occurred: Cmd('git') failed due to: exit code(128) |
Fixing PR: track the progress here.I'm currently fixing this PR to address the following: [Sweep GHA Fix] The GitHub Actions run failed with the following error logs:
An error has occurred: Cmd('git') failed due to: exit code(128) |
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,
|
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.
LGTM, but please bump Minor
version instead of Patch
What
Based on the docs, the following two streams are deprecated and should be deleted:
GET_BRAND_ANALYTICS_ITEM_COMPARISON_REPORT
GET_BRAND_ANALYTICS_ALTERNATE_PURCHASE_REPORT
Recommended reading order
streams.py
source.py
🚨 User Impact 🚨
This is NOT a breaking change as there are no active customers for the streams listed above.
Pre-merge Actions
Updating a connector
Community member or Airbyter
Airbyter
If this is a community PR, the Airbyte engineer reviewing this PR is responsible for the below items.