Skip to content

Conversation

@balazsorban44
Copy link
Member

Fixes #3767

Reasoning 💡

Checklist 🧢

  • Documentation
  • Tests
  • Ready to be merged

Affected issues 🎟

@codecov-commenter
Copy link

codecov-commenter commented Feb 2, 2022

Codecov Report

Merging #3784 (7cf5bf6) into main (f20d679) will not change coverage.
The diff coverage is 0.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #3784   +/-   ##
=======================================
  Coverage   13.03%   13.03%           
=======================================
  Files          92       92           
  Lines        1458     1458           
  Branches      391      393    +2     
=======================================
  Hits          190      190           
  Misses       1254     1254           
  Partials       14       14           
Impacted Files Coverage Δ
src/core/lib/assert.ts 0.00% <0.00%> (ø)
src/lib/logger.ts 64.28% <ø> (ø)
src/providers/twitter.ts 0.00% <0.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f20d679...7cf5bf6. Read the comment docs.

@balazsorban44 balazsorban44 temporarily deployed to Preview February 2, 2022 02:22 Inactive
@github-actions
Copy link

github-actions bot commented Feb 2, 2022

🎉 Experimental release published on npm!

@balazsorban44 balazsorban44 merged commit a7d34f9 into main Feb 2, 2022
@balazsorban44 balazsorban44 deleted the fix/twitter-oauth-2-warning branch February 2, 2022 11:37
mnphpexpert added a commit to mnphpexpert/next-auth that referenced this pull request Sep 2, 2024
…3784)

* fix(providers): properly warn when using Twitter OAuth 2

* refactor(providers): move Twitter OAuth2 warning to `assert`

* fix: use proper warning code

* refactor: only set boolean
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

core Refers to `@auth/core` providers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Twitter OAuth2 warning causing downgrade to OAuth1

3 participants