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

Fix: revert provider type to maintain backward compatibility #240

Merged
merged 5 commits into from
Oct 19, 2021

Conversation

kangmingtay
Copy link
Member

What kind of change does this PR introduce?

  • Adds back the provider field to app_metadata to maintain backward compatibility across client libraries
  • Adds a providers field to store the list of providers used to sign-in for a user

@kangmingtay
Copy link
Member Author

@dshukertjr apologies for the change again, when this PR is merged it will probably break the gotrue dart client library. We decided to revert the breaking change because it was problematic for our users on the gotrue js client library. Going forward, we'll bump the major version on any breaking changes introduced and deprecate the old field properly.

@awalias awalias self-requested a review October 19, 2021 02:59
@awalias awalias merged commit 6628f81 into master Oct 19, 2021
@awalias awalias deleted the fix/revert-provider-type branch October 19, 2021 03:00
@github-actions
Copy link
Contributor

🎉 This PR is included in version 2.1.12 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@dshukertjr
Copy link
Member

Thanks for the heads up @kangmingtay! We haven't made any changes to the Dart library as we were waiting for the official announcement of breaking change being introduced, so everything is cool!

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

Successfully merging this pull request may close these issues.

3 participants