hotfix/cp-10640-add-scheduled-push-option-in-wordpress-plugin

[harkeshkumar196]

Summary by cubic

Adds a “When to send” option in the WordPress CleverPush metabox so editors can schedule push notifications with preset delays or a custom date/time. Addresses Linear CP-10640 by setting scheduledAt on publish or resend based on the selected delay or custom timestamp, with minor UI copy and input handling fixes.

• New Features
  • Added select: Immediately, 15/30/60/120/240 minutes, or Custom date & time; UI shows the picker only for Custom, clears values when switching, and displays a sent message with a “Send again” checkbox when applicable.
  • Server computes scheduledAt from delay or custom value, stores delay in post meta, reuses the same send flow for publish and “Send again,” and still supports the existing scheduled_at input.

^{Written for commit 8e31c05. Summary will update on new commits.}

[cursor]

You have run out of free Bugbot PR reviews for this billing cycle. This will reset on March 12.

To receive reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

[cubic-dev-ai]

No issues found across 1 file

[cubic-dev-ai]

2 issues found across 1 file

Prompt for AI agents (all issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.


<file name="cleverpush.php">

<violation number="1" location="cleverpush.php:1211">
P2: The delay values (15, 30, 60, 120, 240) are duplicated in `publish_post`, `save_post`, and the HTML template. This makes the code harder to maintain and prone to inconsistencies if options change.

Consider defining these as a class constant (e.g., `DELAY_OPTIONS`) and referencing it in all three locations.</violation>

<violation number="2" location="cleverpush.php:1219">
P2: This fallback logic overrides the "Immediately" selection if a scheduled date remains in the hidden input (e.g., due to a JavaScript error or a previous draft).

If the user selects "Immediately" (`cleverpush_notification_delay` is empty), the code shouldn't fall back to `cleverpush_scheduled_at` unless `cleverpush_notification_delay` was missing from the request entirely (legacy compatibility).

If `$_POST['cleverpush_notification_delay']` is present and empty, it should be treated as an explicit "send now" instruction.</violation>
</file>

_{Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.}