Conversation
Contributor
|
Test merged PR on Playground |
Co-authored-by: Sculptor <[email protected]>
Co-authored-by: Sculptor <[email protected]>
Co-authored-by: Sculptor <[email protected]>
Contributor
Step 1Unchecked privacy policy
Current state 
Design While the checkbox is unchecked, the button should be grey. 
_Design of the notification when clicking the greyed out button_
Checked state privacy policy
Current state 
Design When the checkbox is active, it should be yellow instead of red. Use 'real' Ravi instead of logo in the right column
Current state
Design Clicking the X should trigger a warning screen
When a user clicks the X in the top right, it should go to a separate screen asking for confirmation if they really want to close the onboarding wizard. This to prevent from users accidentally dropping out of the onboarding. |
jdevalk
approved these changes
Dec 12, 2025
Member
|
Looks generally good... |
Contributor
✅ Code Coverage Report
🎉 Great job maintaining/improving code coverage! 📊 File-level Coverage Changes (13 files)🆕 New Files
📈 Coverage Improved
📉 Coverage Decreased
ℹ️ About this report
|
aristath
added a commit
that referenced
this pull request
Dec 17, 2025
- Add all step templates: welcome, whats-what, first-task, more-tasks, badges, email-frequency, settings, quit-confirmation - Add form input templates: checkbox, radio - Add task-specific templates: core-blogdescription, core-siteicon, select-locale, select-timezone Note: PHPStan errors in badges.php are false positives - Badges\Monthly class exists but PHPStan can't detect it
aristath
added a commit
that referenced
this pull request
Dec 17, 2025
…g CSS - Apply progress_planner_dashboard_config filter to allow Onboard_Wizard to add wizard configuration - Wizard config will be merged into prplDashboardConfig for React components - Add onboarding.css from PR #714 (1352 lines) Note: PHPStan errors in badges.php are false positives from original PR
aristath
added a commit
that referenced
this pull request
Dec 17, 2025
- Subscribe users via REST API when they choose 'weekly' emails - Proceed immediately when user chooses 'none' (no API call) - Add loading state (isSubscribing) to disable button during subscription - Add error handling with user-friendly error messages - Clear errors when user changes their choice - Use site and timezoneOffset from config with fallbacks Fixes missing email subscription functionality from PR #714 backport. Also fixes PHPCS formatting issues in class-onboard-wizard.php.
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
4 participants
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.
New user onboarding experience, per https://github.com/ProgressPlanner/progress-planner-pro/issues/177