Skip to content

Use SiteInfo to create WpService#1239

Open
crazytonyli wants to merge 3 commits intotrunkfrom
task/unifi-wp-service-constructor
Open

Use SiteInfo to create WpService#1239
crazytonyli wants to merge 3 commits intotrunkfrom
task/unifi-wp-service-constructor

Conversation

@crazytonyli
Copy link
Contributor

Description

There are two separate constructors to create WpService, based on the type of the site. This PR unifies them by using the existing SiteInfo.

I have also updated the Swift wrapper WordPressAPI to hold on to a SiteInfo instance, to simplify the creating WpService API.

@crazytonyli crazytonyli requested a review from jkmassel March 17, 2026 02:19
@wpmobilebot
Copy link
Collaborator

wpmobilebot commented Mar 17, 2026

XCFramework Build

This PR's XCFramework is available for testing. Add to your Package.swift:

.package(url: "https://github.com/automattic/wordpress-rs", branch: "pr-build/1239")

Built from 77c19d3

@crazytonyli crazytonyli force-pushed the task/unifi-wp-service-constructor branch 3 times, most recently from 06327b7 to f95fe31 Compare March 17, 2026 06:35
@crazytonyli crazytonyli force-pushed the task/unifi-wp-service-constructor branch from f95fe31 to 5ae04c5 Compare March 18, 2026 21:01
@crazytonyli crazytonyli force-pushed the task/unifi-wp-service-constructor branch from 5ae04c5 to 77c19d3 Compare March 18, 2026 23:09
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.

2 participants