Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update logic for onboarding redirects #374

Closed
wants to merge 1 commit into from

Conversation

wpscholar
Copy link
Member

This PR utilizes the new Install Checker module to check if a site is a fresh WordPress install and updates the logic for redirects as follows:

  • Always skip onboarding if user isn't an admin
  • Allow manually disabling onboarding (via query param - sets DB option)
  • Allow manually enabling onboarding (via query param - sets DB option)
  • Check if onboarding is explicitly on or off (via DB option)
  • If explicitly on, go ahead and trigger onboarding
  • If explicitly off, bypass onboarding
  • If not explicitly set, run through qualifiers
    • User hasn't completed or exited onboarding already
    • Site is fresh install (use module for this)
    • Coming soon is not enabled
  • If onboarding redirect is triggered at any point, update the DB option to disable onboarding for future visits

@wpscholar wpscholar self-assigned this Nov 28, 2023
@arunshenoy99
Copy link
Member

Thanks @wpscholar, have included these as part of #373. Hence, closing out this PR.

@arunshenoy99 arunshenoy99 deleted the update/onboarding-redirect branch November 30, 2023 03:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants