Project

General

Profile

Development #19459

Dégager django-tables (?)

Added by Frédéric Péters about 2 years ago. Updated over 1 year ago.

Status:
Rejeté
Priority:
Normal
Assignee:
-
Target version:
Start date:
16 Oct 2017
Due date:
% Done:

0%

Patch proposed:
No
Planning:
No

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)

0001-WIP-get-rid-of-django-tables-19459.patch View (5.97 KB) Paul Marillonnet, 30 Oct 2017 06:13 PM

History

#3 Updated by Paul Marillonnet about 2 years ago

Le plus gros du boulot étant de réécrire la classe DocumentTable dans fargo.tables, non ?

#4 Updated by Paul Marillonnet about 2 years ago

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 about 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 about 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 almost 2 years ago

Ça me va de continuer mais faudrait juste ajouter des chaînes localisées.

#8 Updated by Benjamin Dauvergne almost 2 years 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.

Also available in: Atom PDF