Skip to content
This repository has been archived by the owner on Feb 22, 2023. It is now read-only.

[local_auth] support v2 embedding #2299

Merged
merged 14 commits into from
Dec 12, 2019
Merged

Conversation

bparrishMines
Copy link
Contributor

@bparrishMines bparrishMines commented Nov 23, 2019

Fixes flutter/flutter#41841

Tests fail on stable until FlutterLifecycleAdapter lands in stable.

@bparrishMines bparrishMines requested a review from xster December 2, 2019 22:28
@xster
Copy link
Member

xster commented Dec 6, 2019

LGTM, thanks!

@@ -11,6 +11,7 @@ source "$SCRIPT_DIR/common.sh"
check_changed_packages > /dev/null

readonly EXCLUDED_PLUGINS_LIST=(
"flutter_plugin_android_lifecycle"
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Adding this to the exclude list because the all plugins app creates a local dependency while this plugin creates a hosted dependency. This caused errors at compilation.

@bparrishMines bparrishMines merged commit 6bf0290 into flutter:master Dec 12, 2019
@bparrishMines bparrishMines deleted the la_v2 branch December 12, 2019 18:41
@mehmetf
Copy link
Contributor

mehmetf commented Dec 22, 2019

I am confused about this PR. The lifecycle adapter does not exist in master either: https://github.com/flutter/engine/tree/master/shell/platform/android/io/flutter/embedding/engine/plugins/lifecycle.

FlutterSu pushed a commit to FlutterSu/flutter-plugins that referenced this pull request Nov 20, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[local_auth] Support the v2 Android embedder plugins API
4 participants