Bump wordpress-rs to a build exposing unified support conversations#25719
Open
adalpari wants to merge 1 commit into
Open
Bump wordpress-rs to a build exposing unified support conversations#25719adalpari wants to merge 1 commit into
adalpari wants to merge 1 commit into
Conversation
Pin wordpress-rs to the pr-build/1414 build, the first SPM-consumable wordpress-rs build that exposes the combined unified support-conversations endpoints (`unifiedConversations`), introduced in wordpress-rs#1288. The 0.3.0 release this replaces only exposes the separate `supportBots` and `supportTickets` executors. The unified endpoint is required to port the Android unified "Get help" support flow, which serves bot and Happiness Engineer conversations from a single combined list/detail/reply API. This is an interim pr-build pin (same mechanism as the prior integrate/wordpress-rs-pr-1367 work). It should move to a tagged wordpress-rs release before shipping, since no tagged release includes the unified endpoints yet.
Collaborator
Generated by 🚫 Danger |
Contributor
|
| App Name | WordPress | |
| Configuration | Release-Alpha | |
| Build Number | 32934 | |
| Version | PR #25719 | |
| Bundle ID | org.wordpress.alpha | |
| Commit | f6eac56 | |
| Installation URL | 4e1ngncej9ano |
Contributor
|
| App Name | Jetpack | |
| Configuration | Release-Alpha | |
| Build Number | 32934 | |
| Version | PR #25719 | |
| Bundle ID | com.jetpack.alpha | |
| Commit | f6eac56 | |
| Installation URL | 03j7je7dqom7o |
crazytonyli
reviewed
Jul 1, 2026
| // unified support-conversations endpoints (`unifiedConversations`, added in | ||
| // wordpress-rs#1288). No tagged release includes them yet — move to a tagged | ||
| // release before this ships. See PR description. | ||
| branch: "pr-build/1414" |
Contributor
There was a problem hiding this comment.
We can't merge this since the branch goes away after the PR is merged.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.


Why this is needed
We're porting the Android unified "Get help" support flow (WordPress-Android#22946, #23011) to iOS. That flow merges the separate "Ask the Bots" and "Ask the Happiness Engineers" features into a single combined conversations list + type‑branched detail, backed by a new server‑side combined endpoint in
wordpress-rs.iOS today pins
wordpress-rs0.3.0, which only exposes the separate executors:supportBots— bot chatssupportTickets— HE ticketssupportEligibilityIt does not expose the combined
unifiedConversationsexecutor (list / detail / reply for both conversation types) that the whole unification is built on. That endpoint was added in wordpress-rs#1288 ("Add unified support conversations endpoints") — the same rs change Android adopted (1288-85a8554). Without this bump the iOS port is blocked.What changed
Repins
wordpress-rsfromexact: "0.3.0"tobranch: "pr-build/1414"inModules/Package.swift+Modules/Package.resolved.pr-build/1414is the first SPM‑consumablewordpress-rsbuild that includes the unified endpoints. The endpoint landed ontrunk, buttrunk'sPackage.swiftis configured.local(it expects a locally‑built Rust xcframework, which iOS has no tooling to produce), and every existing release tag predates #1288. Thepr-build/<PR#>branches are.release‑configured against a prebuilt xcframework published to the CDN, so SPM can download the binary — exactly the mechanism used previously inintegrate/wordpress-rs-pr-1367.pr-build/1414's only delta over trunk is an unrelated dependabot CI bump that doesn't touch the Swift/Rust surface.Verification
swift package resolvesucceeds and downloads…/wordpress-rs/pr-builds/1414/libwordpressFFI.xcframework.zipfrom the CDN.WPComApiClient.swift) diff from0.3.0→pr-build/1414is purely additive — it only adds theunifiedConversationsexecutor; the existingsupportBots/supportTickets/supportEligibility/meexecutors the app already uses are unchanged. So the bump is expected to be non‑breaking for current call sites.This is an interim
pr-buildpin, not a stable release. Before the unified‑support feature ships,wordpress-rsneeds to cut a tagged release that includes the #1288 unified endpoints, and this dependency should be repinned to that tag (the iOS equivalent of the "move to a tagged release before merge" caveat from the Android PR).pr-build/*branches are not guaranteed to be permanent.🤖 Generated with Claude Code