fix(facebook): Allow latest facebook_flutter_auth#534
fix(facebook): Allow latest facebook_flutter_auth#534aebrahim wants to merge 1 commit intofirebase:mainfrom
Conversation
This fixes firebase#470 by allowing a version of the dependency which adds support for Facebook Limited Login
|
Hello 👋, this PR has been opened for more than 2 months with no activity on it. If you think this is a mistake please comment and ping a maintainer to get this merged ASAP! Thanks for contributing! You have 15 days until this gets closed automatically |
|
@russellwheatley - sorry for the ping, but the github-actions bot told me to ping a maintainer as this PR has gone stale waiting for a review. |
|
Bump! |
|
Bump |
|
BUMP! |
1 similar comment
|
BUMP! |
|
bump |
|
bump! |
|
bump |
1 similar comment
|
bump |
|
Bump |
|
Are you sure it works just upgrade flutter_facebook_auth version? From flutter_facebook_auth 6.x to 7.x there should be breaking change. When I use this PR in my project got error: |
|
Hello 👋, this PR has been opened for more than 2 months with no activity on it. If you think this is a mistake please comment and ping a maintainer to get this merged ASAP! Thanks for contributing! You have 15 days until this gets closed automatically |
|
As an FYI; A PR has been merged that bumps the facebook auth package: #553 On iOS, ATT is taken into account and will use either limited/classic login depending on ATT status. |
Description
This allows a newer version (
7.0.0) of thefacebook_flutter_authdependency which adds support for Facebook Limited LoginRelated Issues
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes (
[x]).This will ensure a smooth and quick review process. Updating the
pubspec.yamland changelogs is not required.///).melos run analyze) does not report any problems on my PR.melos run test:unit:alldoesn't fail).Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?