Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[C2] Add course duration #5633

Closed
ywarnier opened this issue Jul 4, 2024 · 2 comments
Closed

[C2] Add course duration #5633

ywarnier opened this issue Jul 4, 2024 · 2 comments

Comments

@ywarnier
Copy link
Member

ywarnier commented Jul 4, 2024

The idea of evaluating the time necessary to complete a whole course is becoming more and more popular. Despite Chamilo offering the possibility to have extra fields expressing this, it still requires modifications to templates to actually show that information.

Create a duration field of type int in the course table to be future proof. This field can be null.

This field can be edited. If it is set, it should appear in the data returned by api_get_course_info() as duration.
It should also appear (only if it is not null) in the course catalogue and in the list of courses/sessions as a connected user, but it should be set in a specific div class (course-duration) so we can hide it through CSS if we want.

@ywarnier
Copy link
Member Author

ywarnier commented Jul 4, 2024

We also need to add an estimated duration to these:

c_survey.duration
c_quiz.duration (only to display/count time) (does not replace expired_time)
c_quiz_question.duration
c_lp.duration
c_lp_item.duration
c_student_publication.duration (requires migration if previously an extra_field 'work_time' existed attached to c_student_publication)
c_attendance_calendar.duration (requires migration of extra_field 'duration' attached to c_attendance_calendar)

christianbeeznest added a commit to christianbeeznest/chamilo-lms that referenced this issue Jul 5, 2024
christianbeeznest added a commit that referenced this issue Jul 5, 2024
Internal: Add duration field to multiple tables and handle migration of extra fields - refs #5633
@christianbeeznest christianbeeznest removed their assignment Jul 5, 2024
ywarnier added a commit that referenced this issue Jul 5, 2024
@ywarnier
Copy link
Member Author

ywarnier commented Jul 5, 2024

All good from my point of view !

@ywarnier ywarnier closed this as completed Jul 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants