-
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
base: master
Are you sure you want to change the base?
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? |
04e168c
to
a2b4276
Compare
a2b4276
to
99c79c8
Compare
Hi @arielsnowik, I have added the streams and updated the readme. |
Hi @natikgadzhi , @aoelvp94, |
4aa19c8
to
d0ec30d
Compare
Nice, thank you!! As long as the connector is properly tested and there is a clear way (by just reading readme...) for anyone to implement missing endpoints I think it would be ok to merge the PR. Kind Regards |
lgtm, @natikgadzhi and @marcosmarxm how is it possible to continue with this development? |
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?