-
Notifications
You must be signed in to change notification settings - Fork 73
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] Ptitloup/feature improve meeting #817
Conversation
ptitloup
commented
Apr 21, 2023
•
edited
Loading
edited
- add fieldset to show record field
- add settings to specify default slide for meeting session
- add slide field to meeting model
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
f713c90
to
4e23667
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Je n'ai rien vu de choquant. ok pour moi.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Le paramétrage MEETING_PRE_UPLOAD_SLIDES fonctionne parfaitement.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Avec le setting MEETING_PRE_UPLOAD_SLIDES = True
.
Lorsque la réunion ne comporte pas de fichier 'slide', erreur suivante lors de l'ouverture de la réunion :
join() argument must be str or bytes, not 'bool'
De plus, il faudrait ajouter une vérification pour le formulaire d'ajout de réunion pour la taille du nom.
En effet, nous pouvons créer une réunion avec un nom de 1 caractère.
Puis lors de l'ouverture :
Nous ne comprenons pas vraiment comment tester les slides.
Où les slides seront-elles visibles ?
Merci pour ces retours, je vais regarder les problèmes remontés |
Le paramètre MEETING_PRE_UPLOAD_SLIDES doit contenir le chemin vers les slides a afficher par défaut dans une session BBB. Si vous mettez un booleen, cela ne peut pas fonctionner ! :-) |
…/EsupPortail/Esup-Pod into ptitloup/feature_improve_meeting
je viens de rajouter un min validator pour le nom des réunions, merci de me dire @AymericJak si c'est ok avec ca. |
Merci à tous pour vos remarques |
* new organisation for record fields - hide allow start-stop recording, default to True * fix conflicts in translation * add slides field to meeting model * fix conflicts in translation 2 * end of create document url for slides * put url document in https * set limit size to 1 mega bytes * add MEETING_PRE_UPLOAD_SLIDES in configuration * add min length validator for meeting name * put min size for meeting name to 2
* new organisation for record fields - hide allow start-stop recording, default to True * fix conflicts in translation * add slides field to meeting model * fix conflicts in translation 2 * end of create document url for slides * put url document in https * set limit size to 1 mega bytes * add MEETING_PRE_UPLOAD_SLIDES in configuration * add min length validator for meeting name * put min size for meeting name to 2
* new organisation for record fields - hide allow start-stop recording, default to True * fix conflicts in translation * add slides field to meeting model * fix conflicts in translation 2 * end of create document url for slides * put url document in https * set limit size to 1 mega bytes * add MEETING_PRE_UPLOAD_SLIDES in configuration * add min length validator for meeting name * put min size for meeting name to 2
* new organisation for record fields - hide allow start-stop recording, default to True * fix conflicts in translation * add slides field to meeting model * fix conflicts in translation 2 * end of create document url for slides * put url document in https * set limit size to 1 mega bytes * add MEETING_PRE_UPLOAD_SLIDES in configuration * add min length validator for meeting name * put min size for meeting name to 2