-
Notifications
You must be signed in to change notification settings - Fork 2
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
Upgrade flutter, inappwebview, AGP and Android Compile SDK #280
Conversation
7a1d4bc
to
17962c5
Compare
0ae21d6
to
1f4a270
Compare
- Update AGP to 8.2.1 - Bump Flutter SDK lower bound to 3.24.0
There was a problem hiding this 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
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: |
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.
c8435f9
to
b8fd62a
Compare
Added one last change:
@dangeross @danielgranhao We can do a final review. There won't be additional changes on my side in scope of this PR. |
There was a problem hiding this 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
There was a problem hiding this 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.
@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: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
LGTM - thank you for the help @erdemyerebasmaz |
flutter_inappwebview: <6.1.0
seems incompatible with the latest Xcode, failing with several errors similar to:Upgrading forced the upgrade of: