search

EsupPortail / Esup-Pod

plateforme de gestion de fichier vidéo
https://pod.esup-portail.org/
GNU Lesser General Public License v3.0
36 stars 77 forks source link

[DONE] add i18n dynamic url #1135

Open CartierPierre opened 5 months ago

CartierPierre commented 5 months ago

In reference of https://github.com/EsupPortail/Esup-Pod/issues/1130

Tested only on channels, it have to be tested on other impacted URL (video, videos, playlists)

There is no sense to use i18n for hidden or not shared urls

Badatos commented 1 month ago

Cette PR est marquée [WIP], le travail semble donc toujours en cours ? Quand une PR est prête pour la revue, merci de la marquer en "[DONE]".

CartierPierre commented 1 month ago

En soit, c'est codé, je n'ai juste pas pu tout tester, j'ai seulement testé sur les chaines, mais pas sur les vidéos

Badatos commented 1 month ago

En soit, c'est codé, je n'ai juste pas pu tout tester, j'ai seulement testé sur les chaines, mais pas sur les vidéos

dans ce cas, tu peux renommer le [WIP] en [DONE] ? À voir si on trouve une bonne âme pour tester cela du coup...

Badatos commented 1 month ago

Au passage, peux-tu mettre à jour ta branche avec la dernière version de [develop] ?

CartierPierre commented 1 month ago

@Badatos Voilà 😃

Badatos commented 1 month ago

Merci Pierre ;)

Visiblement, il va falloir mettre a jour les tests unitaires pour que cette modif passe.

Il y a par exemple "test_get_channels_for_specific_channel_tab" qui teste /second-channel/ et recoit /en/second-channel/

cf https://github.com/EsupPortail/Esup-Pod/actions/runs/11250448930/job/31279336593?pr=1135 pour le log complet.