[DONE] Raise aristote failure #1382
Conversation
Badatos
left a comment
Badatos
left a comment
There was a problem hiding this comment.
Merci Céline pour cette correction qui sera bien utile ;)
Juste quelques corrections mineures à apporter :
pod/ai_enhancement/utils.py
Outdated
| if not success: | ||
| message = _( | ||
| "Something went wrong with IA improvement on “%(content_title)s”." | ||
| + " Suggestions for improvement can't be available on %(site_title)s." |
There was a problem hiding this comment.
apostrophe simple à remplacer par la version incurvée : can’t au lieu de can't
pod/locale/fr/LC_MESSAGES/django.po
Outdated
| msgstr "Recherchez une vidéo par titre depuis votre tableau de bord" | ||
|
|
||
| #: pod/ai_enhancement/apps.py | ||
| #: pod/ai_enhancement/apps.py:12 |
There was a problem hiding this comment.
Aaaaah ! Beaucoup trop de modifs dans ce fichier .po
Je t'invite à lancer la commande make lang pour générer les fichiers avec les options correctes.
There was a problem hiding this comment.
ah oui je m'étais lancée dans un bon vieux python3 manage.py makemessages. C'est documenté quelque part ce point là. J'ai un peu cherché sans trouver. c'est bien mieux sans les numéros de ligne !
pod/locale/fr/LC_MESSAGES/django.po
Outdated
| "on %(site_title)s." | ||
| msgstr "" | ||
| "Amélioration de l’IA « %(content_title)s » achevée, elle est maintenant " | ||
| "Amélioration de l’AI « %(content_title)s » a été achevée, et est maintenant " |
There was a problem hiding this comment.
En français, c'est bien "IA", et non "AI" ;)
pod/locale/fr/LC_MESSAGES/django.po
Outdated
| #| "“%(content_title)s” was processed by the AI. Suggestions for improvement " | ||
| #| "are available on %(site_title)s." | ||
| msgid "" | ||
| "Something went wrong with IA improvement on “%(content_title)s”. Suggestions " |
There was a problem hiding this comment.
En anglais, c'est "AI" et non "IA".
pod/locale/fr/LC_MESSAGES/django.po
Outdated
| #| "IA improvement “%(content_title)s” has been completed, and is now " | ||
| #| "available on %(site_title)s." | ||
| msgid "" | ||
| "Something went wrong with IA improvement on “%(content_title)s” on " |
pod/locale/fr/LC_MESSAGES/django.po
Outdated
| msgid "Editing group for the enrichment of the video “%(video_title)s”" | ||
| msgstr "" | ||
| "Spécifier les groupes qui peuvent enrichir la vidéo « %(video_title)s »" | ||
| msgstr "Spécifier les groupes qui peuvent enrichir la vidéo" |
There was a problem hiding this comment.
ici il faut remettre le titre de la vidéo comme il était.
Badatos
left a comment
Badatos
left a comment
There was a problem hiding this comment.
Merci pour cette résolution de conflits :)
ps : Mais tu ne dors jamais ... ? 😨
LoicBonavent
left a comment
LoicBonavent
left a comment
There was a problem hiding this comment.
Nickel. Merci Céline.
C'est toujours bizarre ces traductions (ex: {func.name}: Permission denied for user {current_user.pk}.) qui n'ont rien à voir avec les modifications de code...
3 participants
POD ne traite pas les erreurs qu'il reçoit d'Aristote. Cette PR prévient l'utilisateur que l'enrichissement s'est mal passé. C'est une première étape. Il faudrait sans doute aller plus loin et proposer un nouveau "statut" de l'enrichissement parce que pour l'instant POD laisse penser que l'enrichissement est encore en cours.