Skip to content

Conversation

@CartierPierre
Copy link
Contributor

In reference of #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
Copy link
Collaborator

Badatos commented Oct 8, 2024

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
Copy link
Contributor Author

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
Copy link
Collaborator

Badatos commented Oct 8, 2024

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
Copy link
Collaborator

Badatos commented Oct 8, 2024

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

@CartierPierre CartierPierre changed the title [WIP] add i18n dynamic url [DONE] add i18n dynamic url Oct 9, 2024
@CartierPierre
Copy link
Contributor Author

@Badatos Voilà 😃

@Badatos
Copy link
Collaborator

Badatos commented Oct 9, 2024

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.

@Badatos Badatos linked an issue Jan 23, 2025 that may be closed by this pull request
@Badatos Badatos marked this pull request as draft April 23, 2025 12:13
@Badatos Badatos added this to the 3.x milestone Sep 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

i18n dans l'url

3 participants