Skip to content

Conversation

@salimbraksa
Copy link
Contributor

Related PR

The following PR in WordPressKit should be merged first to trigger a pod update.

Description

This PR fixes a build error due to WordPressKit API breaking changes. During the rewrite of RemoteReaderSiteInfo from Objective-C to Swift, the postSubscription and emailSubscription properties are now optional. This PR handles the nullability of those properties.

CleanShot 2022-12-22 at 12 43 13@2x

How to Reproduce

  1. Switch to wpshared-remove-cocoalumberjack branch
  2. Open the Podfile
  3. Install latest WordPressKit changes
def wordpress_kit
  # pod 'WordPressKit', '~> 5.0'
  # pod 'WordPressKit', :git => 'https://github.com/wordpress-mobile/WordPressKit-iOS.git', :tag => ''
  pod 'WordPressKit', :git => 'https://github.com/wordpress-mobile/WordPressKit-iOS.git', :branch => 'trunk'
  # pod 'WordPressKit', :git => 'https://github.com/wordpress-mobile/WordPressKit-iOS.git', :commit => ''
  # pod 'WordPressKit', :path => '../WordPressKit-iOS'
end
  1. Open Xcode and try building the app
  2. Expect the build to fail

Test Instructions:

  1. Switch back to this branch
  2. Install latest WordPressKit changes
def wordpress_kit
  # pod 'WordPressKit', '~> 5.0'
  # pod 'WordPressKit', :git => 'https://github.com/wordpress-mobile/WordPressKit-iOS.git', :tag => ''
  pod 'WordPressKit', :git => 'https://github.com/wordpress-mobile/WordPressKit-iOS.git', :branch => 'trunk'
  # pod 'WordPressKit', :git => 'https://github.com/wordpress-mobile/WordPressKit-iOS.git', :commit => ''
  # pod 'WordPressKit', :path => '../WordPressKit-iOS'
end
  1. Open Xcode and build the app
  2. Expect the build to succeed

Regression Notes

  1. Potential unintended areas of impact
    None.

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

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

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.

@salimbraksa salimbraksa added this to the 21.5 milestone Dec 22, 2022
@salimbraksa salimbraksa self-assigned this Dec 22, 2022
@salimbraksa salimbraksa force-pushed the task/fix-wordpress-kit-compatibilty-issue branch from 4755133 to 5eef2c7 Compare December 22, 2022 12:13
@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 pr19802-5eef2c7 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 pr19802-5eef2c7 on your iPhone

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

@salimbraksa salimbraksa modified the milestones: 21.5, 21.6 Dec 23, 2022
Base automatically changed from wpshared-remove-cocoalumberjack to trunk January 9, 2023 03:10
@jkmassel jkmassel deleted the task/fix-wordpress-kit-compatibilty-issue branch July 26, 2024 18:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants