Open
Conversation
Allow users to schedule posts for future publication from the More actions dropdown menu, using a DateTimePicker popover with timezone display and dynamic Schedule/Publish confirmation button. Closes #31
Contributor
There was a problem hiding this comment.
Pull request overview
Adds post scheduling support to Press This, exposing a “Schedule/Reschedule” action in the header UI and wiring scheduled datetime through the frontend save flow into the REST save endpoint using WordPress’s native future status.
Changes:
- Add a Schedule/Reschedule UI in the Header using
DateTimePickerwithin aPopover, including timezone display and publish-vs-schedule logic. - Extend the save flow to submit an optional
dateparameter and show a schedule-specific snackbar on success. - Update the REST save endpoint to accept
futurestatus + validatedate, setpost_date/post_date_gmt, and gate scheduling behindpublish_postscapability; add PHP/JS tests and styling.
Reviewed changes
Copilot reviewed 12 out of 12 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
press-this-plugin.php |
Accept future status + date param in REST save route and implement scheduling logic in save handler. |
class-wp-press-this-plugin.php |
Thread timezone, postStatus, and postDate into pressThisData for the React app. |
includes/class-press-this-assets.php |
Add wp-keyboard-shortcuts to fallback dependency list. |
src/App.js |
Maintain postStatus/postDate in React state and thread timezone/status/date to Header + Editor. |
src/components/Header.js |
Implement Schedule/Reschedule menu item + DateTimePicker popover workflow and timezone label. |
src/components/PressThisEditor.js |
Send date in REST body, show schedule snackbar, and notify parent of status/date changes. |
src/styles/main.scss |
Import new schedule popover styles. |
src/styles/partials/_header-schedule.scss |
Add styles for schedule popover layout and responsive sizing. |
tests/php/test-scheduling.php |
Add backend tests for scheduling status/date validation, capability gating, and date fields. |
tests/components/header-schedule-ui.test.js |
Add string-based UI coverage for schedule menu/popover/confirm logic. |
tests/components/save-handler-scheduling.test.js |
Add string-based tests for save handler date threading + schedule snackbar behavior. |
tests/components/scheduling-integration.test.js |
Add end-to-end string-based checks across Header → App → Editor save flow. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- Use mysql_to_rfc3339() for postDate to ensure consistent cross-browser parsing - Derive timezone abbreviation from selected date instead of current date (DST accuracy) - Avoid browser-timezone reinterpretation in formatScheduleDate by formatting in UTC - Rename error code to press_this_invalid_date_format for API consistency
Contributor
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 12 out of 12 changed files in this pull request and generated 5 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- Parse date parts in getTimezoneAbbreviation to avoid DST mismatch from browser-local interpretation - Use IANA timezone string directly in formatScheduleDate snackbar instead of deriving abbreviation from wrong instant - Anchor date validation regex to reject trailing timezone qualifiers (Z, -05:00, etc.) - Fix block_http filter accepted_args for PHP 8+ compatibility
- Replace new Date() parsing in isFutureDate with parseNaiveToMs() that extracts date parts directly via Date.UTC(), avoiding browser-local timezone interpretation that breaks at DST boundaries - Use IANA timezone string in formatScheduleDate snackbar (simpler, always correct) - Parse date parts in getTimezoneAbbreviation via Date.UTC() for DST accuracy - Anchor date validation regex to reject trailing tz qualifiers - Fix block_http filter accepted_args for PHP 8+ compatibility
Contributor
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 12 out of 12 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
…dule coverage - Export scheduling utility functions (isFutureDate, parseNaiveToMs, getCurrentDateInTimezone, getTimezoneAbbreviation, formatScheduleDate) as named exports for direct testing - Rewrite all 3 JS test files to call functions with real inputs and assert outputs, replacing fs.readFileSync string-matching approach - Add cross-module integration tests verifying Header utilities produce data consumable by Editor formatting - Add PHP test for rescheduling an already-scheduled post to a new date - Add explanatory comment for edit_date flag in save handler
Contributor
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 12 out of 12 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comment on lines
+112
to
+116
| try { | ||
| const now = new Date(); | ||
| const formatter = new Intl.DateTimeFormat( 'en-CA', { | ||
| timeZone: tz, | ||
| year: 'numeric', |
Intl.DateTimeFormat only accepts IANA zone names and "UTC", not fixed-offset strings like "UTC+2". Add an explicit path that parses the offset and applies it manually. Also normalize the catch fallback to return a naive datetime string (no Z suffix or milliseconds) so scheduled dates stay consistent.
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.
Summary
@wordpress/componentsDateTimePickerin a popover with site timezone displayfuturepost status and wp-cron for automatic publishingCloses #31
Changes
PHP (backend):
press-this-plugin.php: Added'future'to REST status enum,dateparameter with ISO 8601 validation, and scheduling logic in save handler withpublish_postscapability gatingclass-wp-press-this-plugin.php: Addedtimezone,postStatus,postDatetopressThisDataincludes/class-press-this-assets.php: Addedwp-keyboard-shortcutsto fallback dependenciesJS (frontend):
src/components/Header.js: Schedule MenuItem, DateTimePicker Popover with timezone abbreviation, past/future date logic with 1-minute buffer, accessiblearia-labelsrc/components/PressThisEditor.js: ExtendedhandleSaveto pass date, translatable schedule snackbar, locale-aware date formattingsrc/App.js: Threads timezone/capabilities/postStatus/postDate to child components, updates postStatus state after schedulingsrc/styles/partials/_header-schedule.scss: Responsive popover stylesTests:
Test plan
publish_postsfuturestatus, snackbar shows formatted datevendor/bin/phpunit tests/php/test-scheduling.php(5 tests pass)npx jest tests/components/ --no-coverage(17+ tests pass)