Course List: Add new option to show current user's progress #25
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
bernhardkaindl
force-pushed
the
course_list_current_user_progres
branch
from
5fee020 to
a43e417
Compare
|
@ibrahim-kardi @shewa12 @wprashed Please consider merging improvement, thanks! |
bernhardkaindl
force-pushed
the
course_list_current_user_progres
branch
5 times, most recently
from
448b876 to
23ea760
Compare
bernhardkaindl
force-pushed
the
course_list_current_user_progres
branch
from
23ea760 to
a150622
Compare
bernhardkaindl
force-pushed
the
course_list_current_user_progres
branch
from
b4e10f8 to
5a5d327
Compare
Fix crash of Elementor when $authordata is not set, which may be caused by slight inconsistencies of the database. Tutor LMS handles these gracefully, so should do the Elementor addon. For example, when courses were imported but the author was not imported beforehand, $authordata may not be set. Since it would be False and not an array with an ID, this could crash the Elementor editor when opening a page for editing. Fix this crash by checking if $authordata is an array before attempting to use $authordata->ID, and return an empty profile_url in this case.
bernhardkaindl
force-pushed
the
course_list_current_user_progres
branch
2 times, most recently
from
c563f63 to
fa7e1a4
Compare
On the Tutor LMS frontend user dashboard page (site.edu/dashboard), the 1st dashboard entry and the entry "Enrolled courses" show the list of enrolled courses. This plugin's Course List element shows the same course cards. The benefit of this element is of course that it is customizable. Add an option to this plugin's Course List element to enable showing the current user's course progress in the course list's course cards in the same way the dashboard shows the user's course progress for each course. The configuration setting is copied from the "Meta Data" toggle. It defaults to be disabled to only show the user's course progress when explicitly requested to not cause change to existing installations. Additional minor fixes: - Fix spelling of "Course Per Page" to "Courses Per Page" - Fix Elementor Compatibility warning to show the version up to which this plugin is tested with, not "Unknown".
bernhardkaindl
force-pushed
the
course_list_current_user_progres
branch
from
fa7e1a4 to
08fd570
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Hi @ibrahim-kardi!
This Tutor LMS Elementor Addon's Course List element shows course cards for the courses. The benefit of this element over Tutor LMS dashboard is that it is customizable in multiple ways.
4 commits in this PR:
Course List: Add new option to show current user's progress
Course List: Add select to show only enrolled/completed courses
Fix crash of Elementor when
$authordatais not setFix using tutor()->course_post_type and fix typos