chamilo-lms icon indicating copy to clipboard operation
chamilo-lms copied to clipboard

C2: Add migration for links/tools on homepage from C1 to C2

Open ywarnier opened this issue 1 year ago • 7 comments

The migration currently does not deal with links (and other resources) that were added on the courses' homepages in C1 (to the c_tool table). These should now be transformed into "shortcuts" in C2.

Linked to #5073

ywarnier avatar Jan 15 '24 21:01 ywarnier

I'm changing the assignee for approval here, because @NicoDucou will be going through a comprehensive migration soon. This is just a note that there should be a check that links and other resources (like LPs) marked as "show on course homepage" should now be converted to shortcuts when migrated from 1.11 to 2.0.

ywarnier avatar Mar 04 '24 23:03 ywarnier

The link do not appear for the moment on the homepage. It can be checked after migration on this course : https://.........../course/154/home There should be a LP on the homepage.

NicoDucou avatar Mar 26 '24 11:03 NicoDucou

Migration is done for learnpaths and learnpath categories. For links, it was already done.

However, on the functionality side of showing the course home, is needed to filter the learnpath and lp category shortcuts to show then according to the prerequisites between learnpaths

AngelFQC avatar Mar 28 '24 23:03 AngelFQC

However, on the functionality side of showing the course home, is needed to filter the learnpath and lp category shortcuts to show then according to the prerequisites between learnpaths

Related issue #5357

AngelFQC avatar Apr 11 '24 15:04 AngelFQC

Hola Angel, Estuve revisando este issue para validarla pero no me ha funcionado en la migracion de la semana pasada. Revisando me doy cuenta que no se hizo merge al PR https://github.com/chamilo/chamilo-lms/pull/5310 Entonces por eso no estaba incluido en la migracion que hice. Veo que ahora hay conflictos y no se puede hacer merge. Podrias revisarlo y hacer el merge a menos que tengas una razon por no hacer lo ?

NicoDucou avatar Apr 19 '24 14:04 NicoDucou

@NicoDucou acabo de mandar un commit que arregla el conflicto menor. Estaba revisando los PRs, así que lo hice de paso.

ywarnier avatar Apr 19 '24 14:04 ywarnier

I've rebased with master to re-resolve the conflict and change the calls to container and entity manager (post symfony 6 changes)

AngelFQC avatar Apr 23 '24 17:04 AngelFQC