Problem/Motivation

We should add a Tour explaining the features of the checklist module and a guide through the sections.

This could correspond with #3470465: Link the checklist items documentation to the Using SEO Checklist in Drupal CMS guide linking to help documentation if needed.

I have the start of a Tour in Drupal CMS. We can use that to get a jumpstart, then update Drupal CMS to use our Tour.

https://git.drupalcode.org/project/drupal_cms/-/blob/0.x/drupal_cms_seo_...

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

thejimbirch created an issue. See original summary.

thejimbirch changed the visibility of the branch 3473557-add-a-tour to hidden.

thejimbirch’s picture

Status: Active » Needs review

Steps to validate

After checking out this branch, or applying this patch
Enable the SEO Checklist on a site that has the contrib Tour module enabled
Visit the SEO Checklist
Verify the Tour button is available, and clicking the button launches the tour.

traviscarden’s picture

Ooh, interesting! I was sick all last week, so I haven't had a chance to look at this. I'll try to check it out soon.

thejimbirch’s picture

Thanks. Tour was removed from Drupal CMS as it had some bugs in D11. Wanted to move that Tour here to not forget about it!

traviscarden’s picture

Status: Needs review » Fixed

Cool! Fixing a few (mostly grammar) mistakes in the UI text and merging. Thanks!

Status: Fixed » Needs review
traviscarden’s picture

Status: Needs review » Fixed

Oops! I didn't get you credited in the commit message, @thejimbirch. I think it'll pick up on the fact that you authored it. Let's credit you in a comment, too.

  • traviscarden committed 2ae1c830 on 5.2.x
    Revert "Issue #3473557: Add a Tour for SEO Checklist"
    
    This reverts...
traviscarden’s picture

Mmm. I'm not sure that got it. I've just reverted the commit and re-applied it with the correct attribution. Happy Thursday. 😄

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.