Skip to content

Upgrade Android embedding to v2 #25

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

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

ttyniwa
Copy link
Contributor

@ttyniwa ttyniwa commented Oct 2, 2021

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
Copy link
Contributor Author

ttyniwa commented Oct 4, 2021

I tested that this PR works on Android 8.

@gunantosteven
Copy link

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
Copy link

ccfiel commented Dec 21, 2021

@andrey-ushakov can we merge this to master?

@fuadreza
Copy link

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