Wordpress-seo: Reintroduce configuration wizard notice as default notification

Created on 10 Apr 2020  路  6Comments  路  Source: Yoast/wordpress-seo

Up until now, we've had a configuration wizard notice on the dashboard that wasn't included in the notification center:
Schermafbeelding 2020-04-10 om 09 58 52

In https://github.com/Yoast/wordpress-seo/pull/14725 that notification is/will be removed. However, we don't want to remove the notification entirely. It should be brought back as a default notification:

  • It should be included in the notification center under Notifications
  • It should be dismissible
  • The image, checkmark and all other styling can be omitted. Make it a 'bare' notification like all other ones in the notification center.

Base your PR on feature/admin-redesign (like https://github.com/Yoast/wordpress-seo/pull/1472). That way we're sure users will always have a configuration wizard notice.

development

Most helpful comment

@karlijnbok the notifications can be the same! At this moment I am working with UX on the copy and flow of the wizard, so if there are changes to be made we will pick it up at a later moment. It is out op scope for your change...

All 6 comments

We also have another notification already concerning the configuration page ('wpseo-dismiss-onboarding-notice'), see image below:

Screenshot 2020-04-10 at 14 46 23

This notification is triggered when show_onboarding_notice in wpseo in wp_options is 1 AND the configuration wizard is not finished. To manually trigger this notification:

  1. In the database, set show_onboarding_notice (in wpseo in wp_options) to 1
  2. Open the configuration wizard, wait untill it has loaded and click "Close the wizard" Screenshot 2020-04-10 at 14 51 11
  3. The notification should be shown in Notifications

This notification is only removed once you have finished the configuration wizard, i.e., click "close" in the last tab of the configuration wizard:
Screenshot 2020-04-10 at 14 53 06

@IreneStr @JessieHenkes @jdevalk Should these stay separate notifications, or should they be combined?

Please combine them! Btw, you can re-test the notifications a lot simpler with the Yoast test helper plugin, this should have a feature to do just what you said above :)

@jdevalk to double-check whether this was your idea for combining the notifications :)

Me and @JesserH came up with the following idea for the notification:
One notification class with three possible notifications (one of them is shown):

  1. The first time notification, when user has not opened configuration wizard yet
    configuration-first-time (turned into a notification)
  2. The user has opened the configuration wizard, but has not finished the configuration wizard (existing notification) configuration-not-finished
  3. The user has finished the configuration wizard
    configuration-finished (turned into a notification)

Is it correct that only the last notification (3) should be dismissible (besides also being hideable), or should the others also be dismissible? Might also be good to take into account that no other dismissible notifications exist (after wpseo-post-type-archive is removed #14496).

@JessieHenkes Could you provide some feedback regarding the copy :)? Do you want the new notification copies to be the same as in the screenshots in the comment above (https://github.com/Yoast/wordpress-seo/issues/14800#issuecomment-616464749), or do any adjustments need to be made?

@karlijnbok the notifications can be the same! At this moment I am working with UX on the copy and flow of the wizard, so if there are changes to be made we will pick it up at a later moment. It is out op scope for your change...

Was this page helpful?
0 / 5 - 0 ratings