Skip to content

Sync site discussion defaults from Core REST site settings#25542

Open
crazytonyli wants to merge 2 commits intotrunkfrom
task/sync-discussion-defaults-from-rest
Open

Sync site discussion defaults from Core REST site settings#25542
crazytonyli wants to merge 2 commits intotrunkfrom
task/sync-discussion-defaults-from-rest

Conversation

@crazytonyli
Copy link
Copy Markdown
Contributor

Description

This PR applies the "Allow comments" and "Enable Pingback" site settings to the Blog.settings.

The first commit simply formats Swift code. The meaningful changes are in the second commit.

Pre-formatting before fixing the discussion defaults mapping.
Map wordpress-rs `defaultCommentStatus` and `defaultPingStatus` onto `RemoteBlogSettings.commentsAllowed` and `pingbackInboundEnabled` so self-hosted sites' Core Data caches reflect the actual server-side discussion defaults instead of being coerced to false.
@crazytonyli crazytonyli added this to the 26.9 milestone May 8, 2026
@crazytonyli crazytonyli requested a review from jkmassel May 8, 2026 00:48
@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 Number32169
VersionPR #25542
Bundle IDorg.wordpress.alpha
Commit80255a1
Installation URL7lfoc5snp5e20
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 Number32169
VersionPR #25542
Bundle IDcom.jetpack.alpha
Commit80255a1
Installation URL0j1ovjecvt2p8
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