Skip to content
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

Upgrade flutter, inappwebview, AGP and Android Compile SDK #280

Merged
merged 13 commits into from
Jan 6, 2025

Conversation

danielgranhao
Copy link
Contributor

flutter_inappwebview: <6.1.0 seems incompatible with the latest Xcode, failing with several errors similar to:

.pub-cache/hosted/pub.dev/flutter_inappwebviewios-1.0.13/ios/Classes/InAppWebView/InAppWebView.swift:1432:26: error: method does not override any method from its superclass

Upgrading forced the upgrade of:

  • Flutter
  • Android Compile SDK
  • AGP
  • Gradle

@danielgranhao danielgranhao force-pushed the upgrade-inappwebview-flutter-agp-compile-sdk branch from 7a1d4bc to 17962c5 Compare January 3, 2025 14:25
@danielgranhao danielgranhao force-pushed the upgrade-inappwebview-flutter-agp-compile-sdk branch from 0ae21d6 to 1f4a270 Compare January 3, 2025 15:31
@danielgranhao danielgranhao marked this pull request as ready for review January 3, 2025 18:43
@danielgranhao
Copy link
Contributor Author

Builds for iOS and Android are now successful:

- Update AGP to 8.2.1
- Bump Flutter SDK lower bound to 3.24.0
Copy link
Contributor

@dangeross dangeross left a comment

Choose a reason for hiding this comment

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

LGTM. Tested Android

warning: [options] source value 8 is obsolete and will be removed in a future release
warning: [options] target value 8 is obsolete and will be removed in a future release
- Log affected plugins
- Group everything under single subprojects block
@erdemyerebasmaz
Copy link
Collaborator

Thanks for taking the initiative @danielgranhao, changes look good! This was sth I was working on before my leave so I pulled some of my local changes and updates dependencies to latest resolvable versions.

Changelist:

  • chore: update dependencies to latest resolvable ebce3bf
  • chore: update source and target compatibility 785c80d
  • Apply resource linking workaround only on affected plugins d88a862
  • Remove package name from AndroidManifest da15c53
  • Replace the discontinued uni_links plugin with app_links 3e00e24

Remove app_group_directory  & uni_links from main pubspec file as they are not referenced on the app.
Multiple listeners are now allowed on events stream.
@erdemyerebasmaz erdemyerebasmaz force-pushed the upgrade-inappwebview-flutter-agp-compile-sdk branch from c8435f9 to b8fd62a Compare January 6, 2025 12:00
@erdemyerebasmaz
Copy link
Collaborator

Added one last change:

  • Use latest flutter_fgbg from pub.dev b8fd62a
    • instead of our fork on v0.2.0, as multiple listeners are now allowed on events stream with latest published version.

@dangeross @danielgranhao We can do a final review. There won't be additional changes on my side in scope of this PR.

Copy link
Member

@roeierez roeierez left a comment

Choose a reason for hiding this comment

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

LGTM, tested on iOS

Copy link
Contributor

@dangeross dangeross left a comment

Choose a reason for hiding this comment

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

LGTM

- Apply changes to resource linking work around as well.
@erdemyerebasmaz
Copy link
Collaborator

erdemyerebasmaz commented Jan 6, 2025

@roeierez @dangeross Thanks for the swift response.

I have triggered new builds for:

to be certain there are no mandatory changes we need to apply on breez-sdk-liquid.

We can merge once they are both successful:

  • Build iOS
  • Build Android

Copy link
Collaborator

@erdemyerebasmaz erdemyerebasmaz left a comment

Choose a reason for hiding this comment

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

LGTM

@danielgranhao
Copy link
Contributor Author

LGTM - thank you for the help @erdemyerebasmaz

@danielgranhao danielgranhao merged commit fb86737 into main Jan 6, 2025
7 checks passed
@danielgranhao danielgranhao deleted the upgrade-inappwebview-flutter-agp-compile-sdk branch January 6, 2025 13:10
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