Make customize template task point to Learning Mode template

[Imran92]

Resolves #7376

Proposed Changes

• After setting up the site in onboarding flow, clicking on the Customize your theme task took the user to the site editor which sometimes can confuse the user, now we are taking them to edit the Lesson template instead.

Testing Instructions

1. Create a new site with Sensei
2. Install and activate the Course theme
3. In Sensei LMS -> Home, make sure you see the task Customize your lesson template instead of Customize your theme
4. Click on the task
5. Make sure you are taken the Learning Mode's Lesson Template editor page

Before

After

Pre-Merge Checklist

• PR title and description contain sufficient detail and accurately describe the changes
• Acceptance criteria is met
• Decisions are publicly documented
• Adheres to coding standards (PHP, JavaScript, CSS, HTML)
• All strings are translatable (without concatenation, handles plurals)
• Follows our naming conventions (P6rkRX-4oA-p2)
• Hooks (p6rkRX-1uS-p2) and functions are documented
• New UIs are responsive and use a mobile-first approach
• New UIs match the designs
• Different user privileges (admin, teacher, subscriber) are tested as appropriate
• Code is tested on the minimum supported PHP and WordPress versions
• User interface changes have been tested on the latest versions of Chrome, Firefox and Safari
• "Needs Documentation" label is added if this change requires updates to documentation
• Known issues are created as new GitHub issues

[codecov]

Codecov Report

Attention: 6 lines in your changes are missing coverage. Please review.

Comparison is base (4283401) 50.94% compared to head (b612201) 50.94%.
Report is 1 commits behind head on trunk.

Additional details and impacted files

@@            Coverage Diff            @@
##              trunk    #7393   +/-   ##
=========================================
  Coverage     50.94%   50.94%           
- Complexity    11150    11153    +3     
=========================================
  Files           613      613           
  Lines         47063    47067    +4     
  Branches        404      404           
=========================================
+ Hits          23976    23980    +4     
  Misses        22760    22760           
  Partials        327      327           

Files	Coverage Δ
.../class-sensei-home-task-customize-course-theme.php	71.42% <100.00%> (+28.57%)	⬆️
includes/class-sensei-admin.php	19.04% <0.00%> (-0.04%)	⬇️
...ncludes/course-theme/class-sensei-course-theme.php	18.59% <0.00%> (-0.19%)	⬇️

---

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d53e6a4...b612201. Read the comment docs.

[renatho]

It works well, but I think we should also update the completion logic (Sensei_Admin::admin_init) to mark it as completed only when visiting the lesson template.

[Imran92]

It works well, but I think we should also update the completion logic (Sensei_Admin::admin_init) to mark it as completed only when visiting the lesson template.

Thanks @renatho ! I've updated the code here 5d1bf9f . Now it will be marked as completed only when the user clicks the task.

A side-effect of this will be that it'll no longer be marked as completed if the user visits the template in the site editor manually. But that's an edge case I think we can leave for now as there's not a reliable way ATM to detect if the user has visited the template in site editor or not.

[renatho]

Thank you, @Imran92!

Looks good and works well for me!

A side-effect of this will be that it'll no longer be marked as completed if the user visits the template in the site editor manually. But that's an edge case I think we can leave for now as there's not a reliable way ATM to detect if the user has visited the template in site editor or not.

And I think we can live with this side-effect!

[donnapep]

@Imran92 It seems that we're taking the user to the site editor, but not actually opening the template in the editor. Can we make that change? The user shouldn't have to click the pencil icon to get there. We should make it as easy as possible for them. This should make the completion logic simpler too.