search

chamilo / chamilo-lms

Chamilo is a learning management system focused on ease of use and accessibility
https://chamilo.org
GNU General Public License v3.0
798 stars 480 forks source link

Move session image extra field to the session table #4366

Closed ywarnier closed 9 months ago

ywarnier commented 2 years ago

As indicated in #3788, the session extra field for image has been converted to an asset, but this really should become part of the session entity (session table) instead of being an extra field.

christianbeeznest commented 1 year ago

It is moved the session image extra field to the session table as asset in this PR https://github.com/chamilo/chamilo-lms/pull/4588 Thanks for confirmation.

ywarnier commented 1 year ago

Merge incorporado. Falta hacer pruebas en una migración de 1.11 a 2.0

ywarnier commented 9 months ago

@NicoDucou do you have a way to check the maintenance of accessibility of session images (illustration of the session itself) from 1.11 to 2.0?

NicoDucou commented 9 months ago

@NicoDucou do you have a way to check the maintenance of accessibility of session images (illustration of the session itself) from 1.11 to 2.0?

On My sessions page https://CHAMILO/sessions the image is used and on a migrated plateforme we have the session's image presented on the page.

We can also see the image in the session resume page main/session/resume_session.php

So I would say that this is all correct or do you (@ywarnier) think of something different that needs to be checked ?

ywarnier commented 9 months ago

I think it is all correct. Feel free to close.