Skip to content
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

Fixes #56 - update supabase_flutter to 2.0.1 #57

Merged

Conversation

JavascriptMick
Copy link
Contributor

What kind of change does this PR introduce?

(#56)

What is the current behavior?

Please link any relevant issues here.

What is the new behavior?

Feel free to include screenshots if it includes visual changes.

Additional context

Add any other context or screenshots.

@dshukertjr dshukertjr self-requested a review December 16, 2023 02:25
@dshukertjr
Copy link
Member

Thanks for this! It seems like with the latest version of Flutter, the lint rules have changed, causing the format test to fail, but we should be able to fix that in a separate PR! Once those are sorted out, we can publish a new version of supabase Flutter auth UI!

@dshukertjr dshukertjr merged commit 6203cf4 into supabase-community:main Dec 16, 2023
1 check failed
@JavascriptMick JavascriptMick deleted the supabase_flutter_2 branch December 16, 2023 09:18
@JavascriptMick
Copy link
Contributor Author

np @dshukertjr yeah I wasn't sure what happened with the changes to the quotes. at one point I reverted the doubles to singles with vi but I missed one.

@JavascriptMick
Copy link
Contributor Author

I think there is a problem with this change. there is some issue with access to the 'Provider' enum. in my local I needed to change all references to 'Provider' in this file lib/src/components/supa_socials_auth.dart to 'OAuthProvider' which I believe is from the gotrue package. I was able to run the example app locally after this change but I'm really not confident that I tested it properly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants