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)
History
#3 Updated by Paul Marillonnet almost 2 years ago
Le plus gros du boulot étant de réécrire la classe DocumentTable dans fargo.tables, non ?
#4 Updated by Paul Marillonnet almost 2 years ago
- File 0001-WIP-get-rid-of-django-tables-19459.patch View 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.
#5 Updated by Benjamin Dauvergne almost 2 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.
#6 Updated by Frédéric Péters almost 2 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é).
#7 Updated by Benjamin Dauvergne over 1 year ago
Ça me va de continuer mais faudrait juste ajouter des chaînes localisées.
#8 Updated by Benjamin Dauvergne over 1 year ago
- Target version set to 0.23
#9 Updated by Frédéric Péters over 1 year ago
- Target version changed from 0.23 to future
#10 Updated by Benjamin Dauvergne over 1 year ago
- Patch proposed changed from Yes to No
- Status changed from Nouveau to Rejeté
Je ne suis pas convaincu par ce ticket, désolé pour le taf Paul, je rejette.