Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[done] Conversion en masse de video en brouillon dans admin #831

Merged
merged 10 commits into from
May 11, 2023

Conversation

secale
Copy link
Contributor

@secale secale commented May 9, 2023

No description provided.

Copy link
Collaborator

@Badatos Badatos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Merci pour cette PR.
J'ai juste quelques interrogation sur des changement dans les fichiers de langue étonnement plus nombreux que les changement effectifs dans le code.

@@ -7456,63 +7458,6 @@ msgstr "Modifier la période de visualisation"
msgid "Show view statistics for all videos"
msgstr "Afficher les statistiques de visualisation de toutes les vidéos"

#: pod/video/templatetags/video_tags.py:112
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Comment se fait-il que cette partie sur les video_tags soit supprimée ici ?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Il y avait une exclusion d'un répertoire dans la commande de génération des messages, c'est corrigé

msgid "The file extension not in the allowed extension:"
msgstr ""
"Cette extension de fichier n’est pas présente dans les extensions "
"autorisées :"

#: pod/playlist/static/js/playlist.js:91
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pourquoi supprimer ces chaines concernant les listes de lecture ?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

bizarre effectivement.
On vient de vérifier avec Sébastien et sa commande génère un msg pour la ligne 87 mais pas pour la 91 ...
De mon côté ça semble plus correct.
Donc je viens de pousser le fichier modifié

Sebastien Montel and others added 2 commits May 9, 2023 12:28
Copy link
Collaborator

@SebastienCozeDev SebastienCozeDev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@AymericJak et moi avons testé en local et tout est fonctionnel.

Néanmoins, nous pensons qu'il serait intéressant d'ajouter la possibilité de faire l'inverse.

@mattbild mattbild requested a review from Badatos May 10, 2023 09:15
Copy link
Contributor

@ptitloup ptitloup left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok pour moi

Copy link
Collaborator

@SebastienCozeDev SebastienCozeDev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Testé en local et tout est fonctionnel.

Néanmoins, il serait intéressant d'ajouter la possibilité de dé-brouillonner.

@ptitloup ptitloup merged commit b7fc711 into EsupPortail:develop May 11, 2023
@mattbild mattbild deleted the draftvideo branch May 11, 2023 08:36
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.

5 participants