-
Notifications
You must be signed in to change notification settings - Fork 77
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] New 3.4.0 fixes #981
[DONE] New 3.4.0 fixes #981
Conversation
…ding a video, for a super admin, for the owner attribute)
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.
Merci pour ces corrections !
Y'a que pour la correction JS que je mets une remarque.
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 code me semble correct !
Il me reste à tester en local. Je ferai ça un peu plus tard :)
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.
Sur le coup, je n'ai pas touché au fichier webpush.js. Il concerne les notifications : je ne vois pas trop son utilité pour les stats... |
Yess, je sais bien que tu n'y a pas touché, mais comme tu fais du bug fix de la 3.4.0, je trouve ça intéressant de le corriger :) Je suis du même avis, je ne vois pas trop l'utilité des notifications pour les stats :) |
On voit que tu bosses avec Nicolas :) |
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.
En fait @SebastienCozeDev, pour le JQuery de la partie Stats, cela a été laissé ainsi puisqu'une PR sur les stats est en cours. Autant éviter de faire 2 fois le même boulot, puisque JQuery a totalement été retiré dans la PR Stats. |
Concernant l'erreur Javascript non bloquante, il faudrait modifier le fichier webpush.js, qui est fourni par le module django-webpush, pour gérer le fait que subBtn peut être null. A voir si l'on veut par la suite contribuer à ce projet... Pour l'instant, je préfère ne pas toucher. |
Le texte informatif provient du modèle; il y a pas mal de champs qui n'ont pas de texte informatif (disciplines, licence, date d'ajout...). Ce n'est pas trop utile quand c'est clair. Bref, je n'en ai pas ajouté pour Propriétaire. |
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.
Ok, merci à toi pour ta PR.
Bonne fin de journée, et bon week-end :)
* Bugfix for create a thumbnail * Fixed bug preventing change of pages or number of lines in interface * Bugfix for issue EsupPortail#948 (An error appears in the form for adding a video, for a super admin, for the owner attribute) * Adding comments and using getElementById * Manage the background and color of select2 single field, for the dark mode
Fixes 3 bugs for 3.4.0 version:
Bugfix for create a thumbnail.
To check, execute a command like this: "python manage.py create_thumbnail id"
Fixed bug preventing change of pages or number of lines in statistics interface
To check, on a statistics page (/stats_view/), try to change the number of lines or page. Without this bugfix, you can see an "data.filter is not a function" error in the browser console and an icon Loading, and no changes.
Bugfix for issue [BUG] Erreur champs owner lors de l'ajout d'une vidéo #948 (An error appears in the form for adding a video, for a super admin, for the owner attribute).
To check, with a super admin user, upload a video file, wait for encoding to finish (while still on the editing page) and save. Without this bugfix, you can see an "owner - this field is required" error.