search

singingwolfboy / flask-dance

Doing the OAuth dance with style using Flask, requests, and oauthlib.
https://pypi.python.org/pypi/Flask-Dance/
MIT License
1.01k stars 158 forks source link

Remove Twitter #416

Closed daenney closed 1 year ago

daenney commented 1 year ago

In light of the Twitter API shutdown, it's no longer possible to use Twitter Oauth. This removes most code and documentation mentioning Twitter. It's kept in a few places in the docs as its only used for illustration purposes there.

Closes #415

codecov[bot] commented 1 year ago

Codecov Report

Merging #416 (a922a00) into main (85180b9) will decrease coverage by 0.08%. The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #416      +/-   ##
==========================================
- Coverage   93.62%   93.55%   -0.08%     
==========================================
  Files          38       37       -1     
  Lines        1083     1070      -13     
==========================================
- Hits         1014     1001      -13     
  Misses         69       69
Impacted Files Coverage Δ
flask_dance/consumer/base.py 90.90% <ø> (ø)
flask_dance/consumer/oauth1.py 69.31% <ø> (ø)
flask_dance/consumer/oauth2.py 93.54% <ø> (ø)
flask_dance/consumer/requests.py 100.00% <ø> (ø)
flask_dance/consumer/storage/sqla.py 91.81% <ø> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

singingwolfboy commented 1 year ago

Thanks, @daenney! Sorry it took me so long to get to this.

afeezaziz commented 1 year ago

Is it possible to merge twitter back to this? because you can still access twitter just for login at least.