-
Notifications
You must be signed in to change notification settings - Fork 485
Remove TwitterAuthUI because its dependency TwitterKit is deprecated. #650
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
50357ab
to
569d5a1
Compare
DO_NOT_SUBMIT. We will deprecate TwitterAuth first in the next release. This PR will be useful after the next release is out. |
Twitter deprecation PR was just submitted to Firebase SDK. See firebase/firebase-ios-sdk#2663 . Reopen this PR. |
Please hold off review. I will need to implement Twitter auth using OAuth. |
a9716cd
to
5ae3285
Compare
Code is available for review. |
travis is drunk. It failed at the same place as #663 while the latter could always gets passed. As none of our UI tests work right now, I'll just submit this |
Users are encouraged to use OAuth login instead.