Ergonomie de l'admin du BdA

lstephan commented

2020-10-21 18:25:55 +02:00

(Migrated from git.eleves.ens.fr)

Les participants sont ordonnés correctement : d'abord par tirage, puis par nom
quand on modifie un participant/une revente, les champs ForeignKey ne contiennent que les objets associés au bon tirage
quelques fields ont de l'autocomplétion en plus,
on peut archiver un tirage, pour qu'il ne soit plus proposé dans les filtres/les dropdown de l'admin

Fixes #276

- Les participants sont ordonnés correctement : d'abord par tirage, puis par nom - quand on modifie un participant/une revente, les champs `ForeignKey` ne contiennent que les objets associés au bon tirage - quelques fields ont de l'autocomplétion en plus, - on peut archiver un tirage, pour qu'il ne soit plus proposé dans les filtres/les dropdown de l'admin Fixes #276

lstephan commented

2020-10-22 18:54:51 +02:00

(Migrated from git.eleves.ens.fr)

changed the description

lstephan commented

2020-10-22 18:55:02 +02:00

(Migrated from git.eleves.ens.fr)

changed the description

bclement commented

2020-10-22 18:57:38 +02:00

(Migrated from git.eleves.ens.fr)

Pourquoi plusieurs querysets différents ? Pourquoi pas :

        qs = Participant.objects.select_related("user", "tirage")

        if self.instance.pk is not None:
            qs = qs.filter(tirage=self.instance.seller.tirage)

        self.fields["confirmed_entry"].queryset = qs
        self.fields["seller"].queryset = qs
        self.fields["soldTo"].queryset = qs

Pourquoi plusieurs querysets différents ? Pourquoi pas : ```suggestion:-13+0 qs = Participant.objects.select_related("user", "tirage") if self.instance.pk is not None: qs = qs.filter(tirage=self.instance.seller.tirage) self.fields["confirmed_entry"].queryset = qs self.fields["seller"].queryset = qs self.fields["soldTo"].queryset = qs ```

bclement commented

2020-10-22 19:00:19 +02:00

(Migrated from git.eleves.ens.fr)

Raison pour supprimer ? Le ModelSelect2 fait ça tout seul ?

lstephan commented

2020-10-22 19:35:23 +02:00

(Migrated from git.eleves.ens.fr)

changed this line in version 2 of the diff

changed this line in [version 2 of the diff](/klub-dev-ens/gestioCOF/-/merge_requests/469/diffs?diff_id=2212&start_sha=147b8514efb82d9191d6b20c15db5d91db6e7ac7#c45908b588ee57f940e5576626d8e5af3edd9cff_298_288)

lstephan commented

2020-10-22 19:35:24 +02:00

(Migrated from git.eleves.ens.fr)

added 1 commit

f88795a6 - Use same qset for every field

Compare with previous version

added 1 commit <ul><li>f88795a6 - Use same qset for every field</li></ul> [Compare with previous version](/klub-dev-ens/gestioCOF/-/merge_requests/469/diffs?diff_id=2212&start_sha=147b8514efb82d9191d6b20c15db5d91db6e7ac7)

lstephan commented

2020-10-22 19:37:43 +02:00

(Migrated from git.eleves.ens.fr)

Parce qu'il y avait plusieurs querysets différents avant... Mais effectivement, c'est sûrement mieux.

lstephan commented

2020-10-22 19:37:43 +02:00

(Migrated from git.eleves.ens.fr)

ModelSelect2 fait appel à une vue d'autocomplétion, qui render ça de son côté.

`ModelSelect2` fait appel à une vue d'autocomplétion, qui render ça de son côté.

lstephan commented

2020-10-22 19:38:14 +02:00

(Migrated from git.eleves.ens.fr)

resolved all threads

bclement commented

2020-10-22 20:10:19 +02:00

(Migrated from git.eleves.ens.fr)

Ce que je veux dire c'est que si les select_related étaient là, c'est sans doute qu'un jour c'était lent sans. Mon inquiétude c'est qu'on perde cette optimisation dans ModelSelect2 et que ça redevienne lent.

Ce que je veux dire c'est que si les `select_related` étaient là, c'est sans doute qu'un jour c'était lent sans. Mon inquiétude c'est qu'on perde cette optimisation dans `ModelSelect2` et que ça redevienne lent.

bclement commented

2020-10-22 21:17:34 +02:00

(Migrated from git.eleves.ens.fr)

La réponse a l'air d'être : oui, ça va être lent, mais du coup il faut faire ça de façon générique dans shared.Select2QuerySetView (je crois). LGTM du coup.

La réponse a l'air d'être : oui, ça va être lent, mais du coup il faut faire ça de façon générique dans `shared.Select2QuerySetView` (je crois). LGTM du coup.

bclement commented

2020-10-22 21:17:36 +02:00

(Migrated from git.eleves.ens.fr)

approved this merge request

lstephan commented

2020-10-23 10:11:10 +02:00

(Migrated from git.eleves.ens.fr)

resolved all threads

lstephan commented

2020-10-23 10:11:17 +02:00

(Migrated from git.eleves.ens.fr)

mentioned in commit fb1a38cff3