#5544 closed defect (bug) (fixed)
The mySQL query / lookup for the lesson plans on Learn is wrong.
Reported by: | carike | Owned by: | |
---|---|---|---|
Milestone: | Priority: | normal | |
Component: | Learn (learn.wordpress.org) | Keywords: | |
Cc: |
Description
To replicate:
- Open a new browser window;
- Go to https://learn.wordpress.org/lesson-plans/
- Open each page in a new tab.
- Compare and find the duplicates.
Since it appears that the total number of published lesson plans in /wp-admin/ (58) matches the total number of lesson plans available for browsing (9 lesson plans x 6 pages + 4 lesson plans on page 7), this means that some of the published lesson plans are not available for browsing.
The order and which lesson plans are duplicated changes from time to time.
To fix:
The mySQL query should refer to the published lesson plans on the Learn platform.
To my knowledge, it currently does some sort of lookup to GitHub and this is incorrect.
Examples during my testing:
The Web Fonts lesson plan appears on page 3 and on page 4.
Here Theme Troubleshooting appears on page 2 AND on page 3.
What You Can Do With WordPress appears on page 3 and page 4.
Will upload screenshots.
Unfortunately these examples are not exhaustive.
Attachments (6)
Change History (14)
#2
@
4 years ago
- Priority changed from highest omg bbq to normal
Can you please post this report upstream at https://github.com/WordPress/learn?
#3
follow-up:
↓ 8
@
4 years ago
I strongly disagree with setting the priority for a ticket that relates to a quarter of published lesson plans not being available / visible to website visitors to "normal" when the site is supposed to launch tomorrow.
Hugh and Corey have been made aware of this ticket on Slack already. Is there another reason for a GitHub issue? If yes, I'll try to find some time later.
#4
@
4 years ago
Is there another reason for a GitHub issue?
Yes, quoting the description of the GitHub repo:
WordPress.org Learn - The canonical source for the code behind https://learn.WordPress.org
#5
@
4 years ago
Thanks.
If the concern is purely an administrative one, I'll see if I can make some time for it next week.
That having been said, I also strongly disagree with requiring contributors / potential contributors to create third party accounts outside of the WordPress ecosystem in order to publicly report problems with the software, or with the website.
It is the main reason why I strongly advocated for a LMS on Learn, rather than requiring non-technical users to contribute content on GitHub.
Such an approach is a barrier to entry goes against inclusivity goals.
#6
@
4 years ago
@carike, Don't worry about the github account. There is already a related issue on github so I don't think a new one is needed: https://github.com/WordPress/learn/issues/116
I didn't know it was effecting the post type pages as well so it is worse than originally thought. I have cross link back to here as well and requested it be escalated.
#7
@
4 years ago
- Resolution set to fixed
- Status changed from new to closed
This is now fixed by https://github.com/WordPress/learn/pull/175
#8
in reply to:
↑ 3
@
4 years ago
Replying to carike:
Is there another reason for a GitHub issue?
When creating this ticket, you should've also seen a warning that it should be created on GitHub instead.
As you're a Bug Gardener on this trac you get to bypass that warning and create the ticket anyway, most people can't create it.
I might update the warning to make it clear that the reason you're able to create the ticket is due to those extra permissions.
Changing the priority on this ticket because of the significant impact.
https://learn.wordpress.org/lesson-plans/ is currently the only place where front end users can discover / explore lesson plans.
If this was simple duplication, it would be one thing, but the duplicates are substituting published content that is supposed to be there.
When I performed this test, the following lesson plans are duplicated on the list:
Child Themes Twenty Sixteen TWICE
Customizer Taglines THRICE
Customizer: Static Vs. Content Front Page THRICE
Dashboard Icons THRICE
Dashboard Overview TWICE
Managing Comments TWICE
Theme Troubleshooting TWICE
Web Fonts TWICE
What Is A Theme TWICE
Widget Areas TWICE
What is Open Source? TWICE
What You Can Do With WordPress TWICE
This means that 15 published lesson plans that should be discoverable to Learn users are not.
That is 25.8% of currently published courses that are not seen if a new user browses through courses.
Here is the raw data, in the order that the lesson plans appeared:
Line breaks separate pages.