Skip to content

Merge trunk in SPM PR#23393

Merged
jkmassel merged 46 commits intotonyli-xcode-target-authenticatorfrom
task/xcode-targets-merge-trunk
Jul 5, 2024
Merged

Merge trunk in SPM PR#23393
jkmassel merged 46 commits intotonyli-xcode-target-authenticatorfrom
task/xcode-targets-merge-trunk

Conversation

@kean
Copy link
Contributor

@kean kean commented Jun 27, 2024

To test:

Regression Notes

  1. Potential unintended areas of impact

  2. What I did to test those areas of impact (or what existing automated tests I relied on)

  3. What automated tests I added (or what prevented me from doing so)

PR submission checklist:

  • I have completed the Regression Notes.
  • I have considered adding unit tests for my changes.
  • I have considered adding accessibility improvements for my changes.
  • I have considered if this change warrants user-facing release notes and have added them to RELEASE-NOTES.txt if necessary.

Testing checklist:

  • WordPress.com sites and self-hosted Jetpack sites.
  • Portrait and landscape orientations.
  • Light and dark modes.
  • Fonts: Larger, smaller and bold text.
  • High contrast.
  • VoiceOver.
  • Languages with large words or with letters/accents not frequently used in English.
  • Right-to-left languages. (Even if translation isn’t complete, formatting should still respect the right-to-left layout)
  • iPhone and iPad.
  • Multi-tasking: Split view and Slide over. (iPad)

staskus and others added 30 commits March 18, 2024 16:57
TopTotalsCell was calling addRows on every configuration of cell which in turn created and added a hierarchy of UIStackView-based views.

Optimizing TopTotalsCell to only add rows once and then make manipulations on existing rows.
These calls were added together with dynamic type support, however, they slow down layout process of the cell
kean and others added 15 commits June 24, 2024 13:34
File Downloads data can be identical which can result in a rare duplicate diffable data source identifiers crash. Pass a unique identifier to ensure that each file downloads row is treated as unique.
* Support media metadata editing for XML-RPC connected self-hosted sites

- Updated WordPressKit supports editing title, description, and caption of the media via XML-RPC
- XML-RPC API doesn't support editing alt-text

* Support editing media metadata via XMLRPC in MediaService

Media is a type of a post therefore "wp.editPost" can be used to edit media metadata. Note that alternative text cannot be edited due to lack of XML-RPC support https://core.trac.wordpress.org/ticket/58582

* Update RELEASE-NOTES.txt
* Fix announcement card keep showing up after tapping Done (#23384)

* Update app translations – `Localizable.strings`

* Update WordPress metadata translations

* Update Jetpack metadata translations

* Bump version number

---------

Co-authored-by: David Christiandy <1299411+dvdchr@users.noreply.github.com>
@dangermattic
Copy link
Collaborator

1 Error
🚫 PR requires at least one label.
3 Warnings
⚠️ Translation files *.lproj/Localizable.strings should only be updated on a release branch.
⚠️ View files have been modified, but no screenshot or video is included in the pull request. Consider adding some for clarity.
⚠️ This PR is larger than 500 lines of changes. Please consider splitting it into smaller PRs for easier and faster reviews.

Generated by 🚫 Danger

@wpmobilebot
Copy link
Contributor

wpmobilebot commented Jun 27, 2024

WordPress Alpha📲 You can test the changes from this Pull Request in WordPress Alpha by scanning the QR code below to install the corresponding build.
App NameWordPress Alpha WordPress Alpha
ConfigurationRelease-Alpha
Build Numberpr23393-1cf0af0
Version25.1
Bundle IDorg.wordpress.alpha
Commit1cf0af0
App Center BuildWPiOS - One-Offs #10258
Automatticians: You can use our internal self-serve MC tool to give yourself access to App Center if needed.

@wpmobilebot
Copy link
Contributor

wpmobilebot commented Jun 27, 2024

Jetpack Alpha📲 You can test the changes from this Pull Request in Jetpack Alpha by scanning the QR code below to install the corresponding build.
App NameJetpack Alpha Jetpack Alpha
ConfigurationRelease-Alpha
Build Numberpr23393-1cf0af0
Version25.1
Bundle IDcom.jetpack.alpha
Commit1cf0af0
App Center Buildjetpack-installable-builds #9305
Automatticians: You can use our internal self-serve MC tool to give yourself access to App Center if needed.

@kean kean requested a review from jkmassel June 27, 2024 21:55
* Install WordPressUI using SPM

* Remove WordPressShared from Podfile

* Add WordPressShared using SPM

* Fix WordPress compilation

* Fix WordPressKit being embeded in the wrong targets

* Disable some warnings in WordPressKit

* Remove redundant manual linker flags

* Fix WordPressKit tests

* Fix WordPressAuthentificator tests

* Remove Specta and Expecta from Podfile

* Fix WordPressAuthentificator tests by temporary disabling LoginFacadeTests

* Update WordPressAuthenticator so that it could be compliled as an ObjC module again

* Rewrite LoginFacadeTests

* Fix WordPressTests

* Add missing executable_path/../../Frameworks in the share extensions
@jkmassel jkmassel merged commit e98daf4 into tonyli-xcode-target-authenticator Jul 5, 2024
@jkmassel jkmassel deleted the task/xcode-targets-merge-trunk branch July 5, 2024 19:27
@kean kean restored the task/xcode-targets-merge-trunk branch July 5, 2024 19:46
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.

5 participants