Skip to content
This repository has been archived by the owner on Feb 20, 2024. It is now read-only.

Refactor \Sensei_Course_Progress_Widget::widget() #61

Open
lkraav opened this issue Oct 21, 2018 · 0 comments
Open

Refactor \Sensei_Course_Progress_Widget::widget() #61

lkraav opened this issue Oct 21, 2018 · 0 comments
Labels
[Pri] Low Cosmetic issue, low impact, easy workaround. [Type] Maintenance Changes to non-production files

Comments

@lkraav
Copy link

lkraav commented Oct 21, 2018

This method has everything crammed into one. Really hard to follow and work with.

First step would probably be refactoring all the hand-built module/lesson queries into \Sensei_Core_Modules etc methods.
lkraav added a commit to conversionxl/sensei-course-progress that referenced this issue Sep 5, 2019
@lkraav
CXL Institute specific query optimizations [1][2]
369cd32 
[1]: woocommerce#61
[2]: https://trello.com/c/Aokgu76C/161-1-lesson-class-order-incorrect-on-course-page
@donnapep donnapep added [Type] Maintenance Changes to non-production files [Pri] Low Cosmetic issue, low impact, easy workaround. labels Oct 3, 2019
lkraav added a commit to conversionxl/sensei-course-progress that referenced this issue Feb 4, 2023
@lkraav
CXL Institute specific query optimizations [1][2]
5d4a77d 
[1]: woocommerce#61
[2]: https://trello.com/c/Aokgu76C/161-1-lesson-class-order-incorrect-on-course-page
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
[Pri] Low Cosmetic issue, low impact, easy workaround. [Type] Maintenance Changes to non-production files
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@lkraav @donnapep