[DONE] add fields to recording rest response #1193
Conversation
Merge pull request EsupPortail#1119 from EsupPortail/develop
[FREEZE] Develop EsupPortail#3.8.0
[DONE] Develop EsupPortail#3.8.1
[DONE] Update settings.py
|
Bonjour @MichelAimar, |
|
Je ne suis pas du tout dev, je fais du django 1 fois par an lorsqu'on passe les mises à jour et qu'il y a des problèmes. Vérifier le code des autres est un tout autre Taf :) |
|
Je ne comprends pas le test qui a planté. Je dois faire quelque chose ? |
Oui : il faut faire une rapide relecture de la PR #1189 pour que les tests puissent repasser sur toutes les autres PR ;) |
Fix the Pa11y-cy bug, by using grafana/docker-puppeteer
Métadonnées pour le référencement dans https://code.gouv.fr/fr/awesome/. cf https://github.com/publiccodeyml/publiccode.yml (standard publiccode.yml).
|
Si vous pouvez mettre à jour votre branche avec les derniers ajouts de "develop", les tests devraient à nouveau passer normalement |
|
Rien ne me ferait plus plaisir mais j'avoue que je ne sais pas comment faire cela .... Est ce directement sur l'interface ? ou est ce en ligne de commande depuis mon code ? |
Normalement ca doit pouvoir se faire depuis votre dépot : https://github.com/MichelAimar/Esup-Pod/tree/add-fields-to-recording-rest-requests (à faire autant de fois que de branches) |
|
Du coup, il y a 2 erreurs de mise en forme python à corriger : ./pod/playlist/rest_views.py:55:9: E123 closing bracket does not match indentation of opening bracket's line |
|
Bon, il me reste cette erreur sur les 3 PR mais je ne la vois pas : je n'ai que 5 caractères à ces 2 lignes, et l'erreur est au 9eme ... Un peu d'aide ? |
|
La correction de l'erreur en question a été intégrée à la branche "develop". |
|
Ok, mais encore une fois, je fais ça comment ? :( |
Il me semblait avoir déja répondu, mais voici comment faire : Depuis votre dépot sur l'interface Github en ligne : https://github.com/MichelAimar/Esup-Pod/tree/add-fields-to-recording-rest-requests (à faire autant de fois que de branches) |
|
C'est bon, je crois que j'ai trouvé le moyen de mettre votre branche à jour ;) |
|
Finalement, les tests passent. Merci pour cette contribution ;) |
## New functionalities: * Use Whisper to format vtt subtitle (#1187) * Remove BBB module and add sipmediagw feature (#1190) * Add NOTIFY_SENDER boolean parameter (#1192) * Add fields to recording rest response (#1193) * Add return thumbnail url,width and height to oembed (#1194) * Add optional proxy URL for request coming from Aristote (#1218) ## Bugs corrected: * Change all filter_fields = () in filterset_fields = [], as filter_fields is deprecated (#1191) * Fix multi carousel (#1200) * Change regroup videos by theme when ORGANIZE_BY_THEME = True (#1203) * Bug habillage (#1211) * Fix migration script for BBB (#1214) * Improve RSS Feeds (#1215) * Fix BBB meeting deletion link (issue #1216) * No crop on thumbnail (#1217) * Use `get_thumbnail` to serve video thumbnail via caching system, to prevent video folder url to be publicly available (#1221) * Addition of a toolbar in the theme description editor of a chain. (issue #1185) * Correction enabling channels and themes to be assigned to a set of videos (issue #1106) * Manage restricted video access right in playlist ## Accessibility improvements: (#1219) * Modify some redundant title strings, adding the targeted object * Remove some redundant titles * Correct i18n strings * Correct duplicated id * Remove broken aria-label id * Add flatpage title in h1 for accessibility + remove H1 title in legal notice and accessibility statement pages ## Quality of Code: * Minor code formatting * Add missing DocStrings * Upgrade GitGuardian config version
Ajoute les champs date_added et recorder dans les paramètres retournés par l'api rest des "recordings".