sensei icon indicating copy to clipboard operation
sensei copied to clipboard

Published 20 hours ago • verified publisher icon Automattic

Add Quick Links section

Open fjorgemota opened this issue 3 years ago • 1 comments

Fixes #5626

Changes proposed in this Pull Request

  • Add Quick Links section components;

Testing instructions

On a WP installation containing this branch of the Sensei code:

  1. Click on Sensei LMS -> Home;
  2. Verify if the Quick Links section appear in the same way as in the image below;
  3. Verify if, when you click on "install demo course", the label of the link changes accordingly;
  4. Verify if the links work correctly;

Screenshots/Videos

Image of the Quick Links section

fjorgemota avatar Oct 10 '22 23:10 fjorgemota

Codecov Report

Merging #5896 (2690a10) into feature/new-onboarding-experience (05ae0be) will increase coverage by 0.00%. The diff coverage is 42.59%.

Impacted file tree graph

@@                         Coverage Diff                          @@
##             feature/new-onboarding-experience    #5896   +/-   ##
====================================================================
  Coverage                                44.06%   44.07%           
  Complexity                                9196     9196           
====================================================================
  Files                                      446      448    +2     
  Lines                                    32773    32833   +60     
  Branches                                   259      267    +8     
====================================================================
+ Hits                                     14442    14471   +29     
- Misses                                   18141    18165   +24     
- Partials                                   190      197    +7
Impacted Files Coverage Δ
assets/home/sections/quick-links.js 41.50% <41.50%> (ø)
assets/home/link.js 100.00% <100.00%> (ø)
includes/class-sensei-quiz.php 59.21% <0.00%> (-0.14%) :arrow_down:
assets/home/section.js 83.33% <0.00%> (ø)
assets/home/grid.js 75.00% <0.00%> (+25.00%) :arrow_up:

Continue to review full report at Codecov.

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

codecov[bot] avatar Oct 10 '22 23:10 codecov[bot]

I'm merging this even though the @renatho's review was dismissed because I just merged the feature branch back into this branch.

Github was warning about a few conflicts, but...locally, those conflicts didn't happen, so... :shrug:

fjorgemota avatar Oct 19 '22 15:10 fjorgemota