Development #19459
Dégager django-tables (?)
0%
Description
À discuter (eocamp?) mais de mon côté je trouve que django-tables ne se justifie pas pour Fargo. (pour la simplicité, pour la possibilité d'adaptations vers des UI plus sophistiquées)
Files
History
Updated by Paul Marillonnet over 5 years ago
Le plus gros du boulot étant de réécrire la classe DocumentTable
dans fargo.tables
, non ?
Updated by Paul Marillonnet over 5 years ago
- File 0001-WIP-get-rid-of-django-tables-19459.patch 0001-WIP-get-rid-of-django-tables-19459.patch added
- Patch proposed changed from No to Yes
Un petit peu de temps pour commencer à regarder comment je pourrais m'y prendre.
J'ai un affichage minimaliste sans dépendance à django_tables2.
Je pose le patch WIP.
Updated by Benjamin Dauvergne about 5 years ago
Je ne comprends pas bien la fixette sur django-tables2, c'est lié à la migration Django 1.11 ? Parce qu'on s'en sert aussi dans authentic en fait.
Updated by Frédéric Péters about 5 years ago
[...] fixette sur django-tables2
Ce module, la compréhension qu'il demande en plus, pouvait être imaginé comme une raison freinant les contributions. (mais comme noté plus haut, ça n'a pas particulièrement été confirmé).
Updated by Benjamin Dauvergne almost 5 years ago
Ça me va de continuer mais faudrait juste ajouter des chaînes localisées.
Updated by Benjamin Dauvergne almost 5 years ago
- Status changed from Nouveau to Rejeté
- Patch proposed changed from Yes to No
Je ne suis pas convaincu par ce ticket, désolé pour le taf Paul, je rejette.