[DONE] add fields to recording rest response

[MichelAimar]

Ajoute les champs date_added et recorder dans les paramètres retournés par l'api rest des "recordings".

[Badatos]

Bonjour @MichelAimar, Merci pour ton implication dans Esup-Pod :) En ce moment, on manque de relecteurs pour relire et tester les différentes PR. Je t'invite à participer par exemple en faisant une relecture de cette PR assez rapide : #1189

[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 :)

[MichelAimar]

Je ne comprends pas le test qui a planté. Je dois faire quelque chose ?

[Badatos]

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 ;)

[Badatos]

Si vous pouvez mettre à jour votre branche avec les derniers ajouts de "develop", les tests devraient à nouveau passer normalement

[MichelAimar]

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 ?

[Badatos]

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 Il doit y avoir un bouton "SYNC FORK" > "update fork", un truc comme ca.

(à faire autant de fois que de branches)

[Badatos]

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 ./pod/video/rest_views.py:214:9: E123 closing bracket does not match indentation of opening bracket's line

[MichelAimar]

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 ?

[Badatos]

C'est une erreur qui a été corrigée dans ce commit https://github.com/EsupPortail/Esup-Pod/pull/1200/commits/f8141aec99ab914c5f421dfab98325c8c39899fb Il suffira donc soit d'attendre que la PR #1200 soit fusionnée, et simplement mettre à jour votre branche, soit d'appliquer les modifications de ce commit à la votre (apparament il s'agit juste d'espaces à retirer)

[Badatos]

La correction de l'erreur en question a été intégrée à la branche "develop". Si vous pouvez mettre à jour votre branche avec les derniers ajouts, les tests devraient à nouveau passer normalement sur votre branche.

[MichelAimar]

Ok, mais encore une fois, je fais ça comment ? :(

[Badatos]

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 Il doit y avoir un bouton "SYNC FORK" > "update fork".

(à faire autant de fois que de branches)

[Badatos]

C'est bon, je crois que j'ai trouvé le moyen de mettre votre branche à jour ;)

[Badatos]

Finalement, les tests passent. Merci pour cette contribution ;)