[DONE] change all filter_fields = () in filterset_fields = [] #1191
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, Pouvez-vous suivre les pré-requis de ce dépôt svp :
|
|
Ah zut, désolé, j'ai suivi une doc mais c'était celle de Esup Portal :( |
MichelAimar
changed the title
|
Voila je pense avoir modifié le PR ... Par contre les tests pour la 3.10 n'ont pas l'air de passer mais je ne sais pas si c'est important ... |
|
Merci pour les corrections. |
|
Bin, elle y était dans le code avant .... je n'en ai aucune idée .... |
|
Python autorise les virgules en debut et fin de liste, c'est donc syntaxiquement correct mais comme cette dernière virgule n'apparait pas en fin des listes précédentes ma remarque visait juste à homogénéiser le style (mais bon c'est du tatillonage ;) |
There was a problem hiding this comment.
Merci pour cette PR !
En effet, comme on peut le voir ici https://django-filter.readthedocs.io/en/stable/guide/migration.html#migrating-to-2-0
Les filter_fields de django-filter > 2.0 sont deprecated depuis 2018-7-13, au profit des nouveaux filterset_fields, et ont été complètement retirés en 22.1 (2022-6-17)
Il est temps en effet de corriger cela ;)
cf https://github.com/carltongibson/django-filter/blob/main/CHANGES.rst
## 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
… filter_fields is deprecated.
Before sending your pull request, make sure the following are done
developbranch.[WIP]or[DONE].