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

tech/update-inappwebview #223

Merged
merged 4 commits into from
Nov 1, 2023
Merged

tech/update-inappwebview #223

merged 4 commits into from
Nov 1, 2023

Conversation

kackogut
Copy link
Collaborator

@kackogut kackogut commented Oct 29, 2023

Description

App was not building properly for iOs build because of incompatible version of WebView library fixed in 5.8.0: pichillilorenzo/flutter_inappwebview#1807

Checklist:

Creator

  • The target branch is dev
  • I have updated the unreleased section of the change log
  • I have reviewed the 'files changed' tab on github to ensure all changes are as expected
  • I have added someone to review the pr

Reviewer

  • I have verified the above are all completed
  • I have run the code locally to ensure it fufills the requirements of the issue
  • I have reviewed the 'files changed' and commented on any sections which I think are not needed, incorrect or could be improved

After pull

  • If appropriate I have closed the issue/ moved the trello card

@kackogut kackogut requested a review from JElgar October 29, 2023 10:43
@kackogut kackogut changed the title Fix iOs builds by upgrading webview version tech/update-inappwebview Oct 29, 2023
Copy link
Member

@JElgar JElgar left a comment

Choose a reason for hiding this comment

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

Thank you!

@kackogut kackogut merged commit edbf272 into dev Nov 1, 2023
@kackogut kackogut deleted the fix_ios_builds branch November 1, 2023 18:07
JElgar added a commit that referenced this pull request Nov 12, 2023
* Update secret extractor

* Fix app build

* Split release workflows

* Remove google services generation

* Fixes - launch learning resource when logged out, new filter for all tab, remove campaign description new line characters

* Fix all analyze warnings (#224)

* Fix all analyze warnings

* Regenerated pod lockfile

* Don't fail analyze on infos

* Fix analysis_options sections

* Regenerated pod lockfile

* Fix nullability issues

* Regenerated pod lockfile

* Remove unrecognized `--set-exit-if-change` from Flutter formatter

---------

Co-authored-by: kackogut <[email protected]>

* tech/update-inappwebview (#223)

* Fix iOs builds by upgrading webview version

* Regenerated pod lockfile

* Regenerated pod lockfile

---------

Co-authored-by: kackogut <[email protected]>

* Regenerated pod lockfile

---------

Co-authored-by: James Elgar <[email protected]>
Co-authored-by: kackogut <[email protected]>
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.

2 participants