Skip to content

Normalize remote post site IDs#25773

Open
crazytonyli wants to merge 2 commits into
trunkfrom
fix/remote-post-site-id-parsing
Open

Normalize remote post site IDs#25773
crazytonyli wants to merge 2 commits into
trunkfrom
fix/remote-post-site-id-parsing

Conversation

@crazytonyli

Copy link
Copy Markdown
Contributor

The site_ID in WP.com rest/1.x API response may be a string or a number. I noticed this issue in testing publishing from the share sheet.

To test:

  1. From Xcode, build and run WordPressShareExtension with the Photos app.
  2. Share a photo -> tap the WordPress icon -> complete the form to publish the post
  3. Wait for the request to complete in the background.
  4. Xcode shows a crash at DDLog

@crazytonyli crazytonyli enabled auto-merge July 12, 2026 23:50
@wpmobilebot

Copy link
Copy Markdown
Contributor
App Icon📲 You can test the changes from this Pull Request in WordPress by scanning the QR code below to install the corresponding build.
App NameWordPress
ConfigurationRelease-Alpha
Build Number33112
VersionPR #25773
Bundle IDorg.wordpress.alpha
Commit3f2865f
Installation URL79c5okseb0h4g
Automatticians: You can use our internal self-serve MC tool to give yourself access to those builds if needed.

@wpmobilebot

Copy link
Copy Markdown
Contributor
App Icon📲 You can test the changes from this Pull Request in Jetpack by scanning the QR code below to install the corresponding build.
App NameJetpack
ConfigurationRelease-Alpha
Build Number33112
VersionPR #25773
Bundle IDcom.jetpack.alpha
Commit3f2865f
Installation URL6vtbbo33re0k0
Automatticians: You can use our internal self-serve MC tool to give yourself access to those builds if needed.

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.

2 participants