Skip to content

Upgrade Android embedding to v2#25

Open
ttyniwa wants to merge 8 commits into
wwandreww:masterfrom
ttyniwa:feature/refactoring-for-android-gradle-plugin-upgrade
Open

Upgrade Android embedding to v2#25
ttyniwa wants to merge 8 commits into
wwandreww:masterfrom
ttyniwa:feature/refactoring-for-android-gradle-plugin-upgrade

Conversation

@ttyniwa

@ttyniwa ttyniwa commented Oct 2, 2021

Copy link
Copy Markdown
Contributor

this PR solves #23

  • upgrade gradle to 7.0.2
  • Upgradle Android Flutter plugin to v2.
  • Fixed some warnings in AndroidStudio
  • Reformat Code by using AndroidStudio

This PR works on Android 8.
I'm new to Flutter Plugin development and Android development, so there may be some oddities.
so, If the PR is not good enough, I don't mind if you close it.

References.
https://flutter.dev/docs/development/packages-and-plugins/plugin-api-migration
https://flutter.dev/docs/development/packages-and-plugins/developing-packages#plugin

@ttyniwa ttyniwa changed the title Upgrade Android embedding to v2 #23 Upgrade Android embedding to v2 Oct 2, 2021
… officially.

> The scripts generated by this task are intended to be committed to your version control system. This task also generates a small gradle-wrapper.jar bootstrap JAR file and properties file which should also be committed to your VCS. The scripts delegates to this JAR.
> https://docs.gradle.org/current/dsl/org.gradle.api.tasks.wrapper.Wrapper.html
@ttyniwa

ttyniwa commented Oct 4, 2021

Copy link
Copy Markdown
Contributor Author

I tested that this PR works on Android 8.

@gunantosteven

Copy link
Copy Markdown

Hi @ttyniwa I tried ur code but got an error "lambda expressions are not supported in -source 7 vscode"

I think using lambda not working for me when building the app.

@ccfiel

ccfiel commented Dec 21, 2021

Copy link
Copy Markdown

@andrey-ushakov can we merge this to master?

@fuadreza

Copy link
Copy Markdown

can this PR be merged?

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.

4 participants