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

Merge finalized release 21.8 (take 2) #20254

Merged
merged 26 commits into from
Mar 6, 2023

Conversation

mokagio
Copy link
Contributor

@mokagio mokagio commented Mar 6, 2023

This takes over #20243, to track the crash fix that landed on 21.8 at the last minute, resulting in the finalized build being rejected and in a new one being created.

See also p8Qyks-53V-p2#comment-1967

dvdchr and others added 26 commits February 24, 2023 22:13
…21.7.2-into-release-21.8

Got conflicts in the `config/Version.*.xcconfig` files. Resolved them,
as usual for hotfixes going into release branches, via `git checkout
--theirs` to keep the release branch version of the files.
…elease_1.89.1

Integrate gutenberg-mobile release 1.89.1
This is a workaround to prevent the crash happening in #20245. The crash happens when the app attempts to clean up Quick Start tours. The clean up is done when the app determines that Jetpack features are no longer to be shown. However, the clean up produces a Core Data crash which I haven't understood properly yet. So a workaround is to simply not do the clean up and leave the Quick Start tours in Core Data. They won't be visible to the user.
Got conflicts in `Podfile` and `Podfile.lock` because both branches
updated Gutenberg.

I addressed them with `git checkout --theirs` to keep the version from
`trunk`, then run `bundle exec pod install` to ensure the lockfile was
entirely up to date.
@mokagio mokagio self-assigned this Mar 6, 2023
@peril-wordpress-mobile
Copy link

Messages
📖 This PR has the 'Releases' label: some checks will be skipped.

Generated by 🚫 dangerJS

@mokagio
Copy link
Contributor Author

mokagio commented Mar 6, 2023

I'm going to admin merge this PR, on the basis that the diff is basically the same as the already approved #20243 and that I need to move on with the code freeze for 21.9.

@mokagio mokagio marked this pull request as ready for review March 6, 2023 02:15
@wpmobilebot
Copy link
Contributor

You can test the changes in Jetpack from this Pull Request by:
  • Clicking here or scanning the QR code below to access App Center
  • Then installing the build number pr20254-302bcab on your iPhone

If you need access to App Center, please ask a maintainer to add you.

@wpmobilebot
Copy link
Contributor

You can test the changes in WordPress from this Pull Request by:
  • Clicking here or scanning the QR code below to access App Center
  • Then installing the build number pr20254-302bcab on your iPhone

If you need access to App Center, please ask a maintainer to add you.

@mokagio mokagio merged commit 583a33d into trunk Mar 6, 2023
@mokagio mokagio deleted the merge/release-21.8-finalized-into-trunk-take-2 branch March 6, 2023 03:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants