Skip to content

Fix missing params in new draft#25416

Open
crazytonyli wants to merge 3 commits intotrunkfrom
bugfix/new-draft-status
Open

Fix missing params in new draft#25416
crazytonyli wants to merge 3 commits intotrunkfrom
bugfix/new-draft-status

Conversation

@crazytonyli
Copy link
Contributor

@crazytonyli crazytonyli commented Mar 20, 2026

Description

Fix two issues:

  • Create a new draft with 'pending review' status. The status is lost after saving.
  • Create a new draft with a future date. The future date is lost after saving.
  • Updating post format did not work. This issue exists on the AbstractPost too.

@crazytonyli crazytonyli added this to the 26.8 milestone Mar 20, 2026
@crazytonyli crazytonyli requested a review from kean March 20, 2026 01:45
@wpmobilebot
Copy link
Contributor

wpmobilebot commented Mar 20, 2026

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 Number31654
VersionPR #25416
Bundle IDorg.wordpress.alpha
Commitee5fff0
Installation URL4q1en4ukgoeog
Automatticians: You can use our internal self-serve MC tool to give yourself access to those builds if needed.

@wpmobilebot
Copy link
Contributor

wpmobilebot commented Mar 20, 2026

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 Number31654
VersionPR #25416
Bundle IDcom.jetpack.alpha
Commitee5fff0
Installation URL1r5j9jni2pl20
Automatticians: You can use our internal self-serve MC tool to give yourself access to those builds if needed.

@wpmobilebot wpmobilebot modified the milestones: 26.8, 26.9 Mar 20, 2026
@wpmobilebot
Copy link
Contributor

Version 26.8 has now entered code-freeze, so the milestone of this PR has been updated to 26.9.

@crazytonyli crazytonyli force-pushed the bugfix/new-draft-status branch from 0111ac0 to ee5fff0 Compare March 20, 2026 06:29
@sonarqubecloud
Copy link

status = .draft
publishDate = nil
if let paramStatus = params.status {
status = BasePost.Status(paramStatus)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It might be worth comparing with the current implementation. The main challenges I encountered were:

  • Anything related to the scheduled posts
  • Remove featured image

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