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

Migrate independent plugins to AndroidX #1103

Merged
merged 3 commits into from
Jan 24, 2019
Merged

Migrate independent plugins to AndroidX #1103

merged 3 commits into from
Jan 24, 2019

Conversation

mklim
Copy link
Contributor

@mklim mklim commented Jan 23, 2019

The original Android support libraries have been deprecated in favor of
AndroidX. This migration shouldn't affect behavior in a user noticeable
way, but does require plugin users to also migrate their app. See
migrating to AndroidX.

Migrates all of our plugins without pub interdepencies to use the new
AndroidX support libraries. At least one followup PR will be required to
update the remaining plugins that depend on these after this is pushed
to pub.

  • android_intent
  • battery
  • connectivity
  • device_info
  • firebase_auth
  • firebase_core
  • google_maps_flutter
  • google_sign_in
  • image_picker
  • in_app_purchase
  • local_auth
  • location_background
  • package_info
  • path_provider
  • quick_actions
  • sensors
  • share
  • shared_preferences
  • url_launcher
  • video_player
  • webview_flutter

flutter/flutter#23995

The original Android support libraries have been deprecated in favor of
AndroidX. This migration shouldn't affect behavior in a user noticeable
way, but does require plugin users to also migrate their app. See
[migrating to AndroidX](https://developer.android.com/jetpack/androidx/migrate).

Migrates all of our plugins without pub interdepencies to use the new
AndroidX support libraries. At least one followup PR will be required to
update the remaining plugins that depend on these after this is pushed
to pub.

- android_intent
- battery
- connectivity
- device_info
- firebase_auth
- firebase_core
- google_maps_flutter
- google_sign_in
- image_picker
- in_app_purchase
- local_auth
- location_background
- package_info
- path_provider
- quick_actions
- sensors
- share
- shared_preferences
- url_launcher
- video_player
- webview_flutter
@mklim mklim changed the title [WIP] Migrate plugins to androidx Migrate independent plugins to AndroidX Jan 24, 2019
@mklim mklim requested review from amirh and dnfield January 24, 2019 20:28
@dnfield
Copy link
Contributor

dnfield commented Jan 24, 2019

I tested this locally using AndroidX plugin with current Flutter and it looks good.

Should we bump the gradle plugins while we're at it to 3.3.0? As in flutter/flutter#26913

It looks like this all works and builds, so maybe we could do that as a separate patch.

Either way, LGTM

@jonahwilliams
Copy link
Member

Does this conflict with our mandate to keep plugins working with the last stable flutter release?

@mklim
Copy link
Contributor Author

mklim commented Jan 24, 2019

Does this conflict with our mandate to keep plugins working with the last stable flutter release?

I've spot checked some of the example apps with flutter stable and what I've seen appears to be working fine. The CI failure on the latest commits looks like something unrelated to this PR, I'm also seeing it on another PR that only updated markdown documentation and it's referencing a problem with /home/cirrus/sdks/flutter/packages/flutter_tools/gradle/flutter.gradle.

@dnfield
Copy link
Contributor

dnfield commented Jan 24, 2019

@jonahwilliams this appears to be backwards compatible (plugins using AndroidX seem to work with Flutter using older support libs), although in the reverse direction it seems to not be (e.g. if someone is using AndroidX in their app, they can't use plugins on the older support libs).

@mklim mklim merged commit 59e84d8 into flutter:master Jan 24, 2019
@mklim mklim deleted the androidx branch January 24, 2019 21:59
bbedward added a commit to bbedward/plugins that referenced this pull request Jan 30, 2019
andreidiaconu pushed a commit to andreidiaconu/plugins that referenced this pull request Feb 17, 2019
The original Android support libraries have been deprecated in favor of
AndroidX. This migration shouldn't affect behavior in a user noticeable
way, but does require plugin users to also migrate their app. See
[migrating to AndroidX](https://developer.android.com/jetpack/androidx/migrate).

Migrates all of our plugins without pub interdepencies to use the new
AndroidX support libraries. At least one followup PR will be required to
update the remaining plugins that depend on these after this is pushed
to pub.

- android_intent
- battery
- connectivity
- device_info
- firebase_auth
- firebase_core
- google_maps_flutter
- google_sign_in
- image_picker
- in_app_purchase
- local_auth
- location_background
- package_info
- path_provider
- quick_actions
- sensors
- share
- shared_preferences
- url_launcher
- video_player
- webview_flutter
andreidiaconu added a commit to andreidiaconu/plugins that referenced this pull request Feb 17, 2019
julianscheel pushed a commit to jusst-engineering/plugins that referenced this pull request Mar 11, 2020
The original Android support libraries have been deprecated in favor of
AndroidX. This migration shouldn't affect behavior in a user noticeable
way, but does require plugin users to also migrate their app. See
[migrating to AndroidX](https://developer.android.com/jetpack/androidx/migrate).

Migrates all of our plugins without pub interdepencies to use the new
AndroidX support libraries. At least one followup PR will be required to
update the remaining plugins that depend on these after this is pushed
to pub.

- android_intent
- battery
- connectivity
- device_info
- firebase_auth
- firebase_core
- google_maps_flutter
- google_sign_in
- image_picker
- in_app_purchase
- local_auth
- location_background
- package_info
- path_provider
- quick_actions
- sensors
- share
- shared_preferences
- url_launcher
- video_player
- webview_flutter
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.

4 participants