-
Notifications
You must be signed in to change notification settings - Fork 3.7k
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 Appsflyer: Migrate to V2 Appsflyer API #38339
🚨 Source Appsflyer: Migrate to V2 Appsflyer API #38339
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
@AnjayGoel thank you for contributing! Running CI on this now. I'd like to remove the dockerfile and setup.py while we're here — would you mind rebasing if/when I ping you? |
@marcosmarxm, this one needs a breaking change migration guide:
Can you link breaking change migration guide docs so @AnjayGoel can write one? @AnjayGoel, this also needs version to be set in both |
# Conflicts: # airbyte-integrations/connectors/source-appsflyer/metadata.yaml # docs/integrations/sources/appsflyer.md
@natikgadzhi I have made the changes |
Triggered CI Tests. |
Hello your feedback matters a lot to us. Can you spare just 3 minutes to complete a survey? We're dedicated to making the contributor experience even better, and your input is key to achieving excellence. Thank you for helping us improve! |
What
Migrate from obsolete V1 API to V2.
Fixes #37403, #35667, #32446.
How
Appsflyer has deprecated & removed the V1 APIs. This PR migrates the connector to the API V2.
Review guide
User Impact
Users will have to replace the API V1 tokens with API V2 tokens as described here.
Can this PR be safely reverted and rolled back?