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.

Sign up for GitHub

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

Jump to bottom

Bug 10 - Argument #2 ($user) must be of type Laravel\Socialite\Two\User #11

Merged

stevebauman merged 2 commits into master from bug-10

Jun 4, 2024

Member

stevebauman commented Jun 4, 2024 •

edited

Loading

Closes #10

This PR fixes the Bartender contracts to use the proper base Socialite User interface instead of the Two\User class:

- use Laravel\Socialite\Two\User as SocialiteUser;
+ use Laravel\Socialite\Contracts\User as SocialiteUser;

This would normally be a breaking change, but I believe this is more so a bug and can thus be placed in a minor update.

stevebauman added 2 commits

June 4, 2024 17:10


          Use socialite user contract instead of hard-coded Two\User

65d62a7


          Remove unused import

868a510

stevebauman merged commit e3baa25 into master

14 checks passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet