Projet

Général

Profil

Development #19033

Pagination dans la liste des exceptions

Ajouté par Frédéric Péters il y a plus de 6 ans. Mis à jour il y a plus de 4 ans.

Statut:
Fermé
Priorité:
Bas
Assigné à:
Catégorie:
-
Version cible:
-
Début:
27 septembre 2017
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

J'ai importé des exceptions depuis un fichier de congés et mon écran est devenu trop petit (cf capture).

Sans doute que la popup devrait être limitée à max 10 éléments, et contenir un lien vers une page listant vraiment toutes les exceptions, avec pagination, filtre, etc. à voir.


Fichiers


Demandes liées

Lié à Chrono - Development #36167: Lever la limite de 600 jours lors de l'import des exceptionsRejeté16 septembre 2019

Actions

Révisions associées

Révision b5ef493d (diff)
Ajouté par Lauréline Guérin il y a plus de 4 ans

time exceptions: never show past exceptions (#19033)

Révision c6f418dd (diff)
Ajouté par Lauréline Guérin il y a plus de 4 ans

time exceptions: paginated list of exceptions (#19033)

Révision 49eff910 (diff)
Ajouté par Lauréline Guérin il y a plus de 4 ans

time exceptions: stay on exception list on delete (#19033)

Historique

#1

Mis à jour par Thomas Noël il y a plus de 6 ans

J'ai un question un peu différente : quel intérêt de présenter ça en popup ? Parce que dans la vraie vie, le nombre d'exception va être important.

#2

Mis à jour par Frédéric Péters il y a plus de 6 ans

L'idée était d'avoir un premier niveau, directement dans la page, les prochaines exceptions à venir, ensuite la popup, avec une plus longue liste, mais pas trop longue, genre pour la situation "tiens je ne sais plus si j'ai noté que ce guichet fermait à Noël".

En vrai, surtout, ça permettait aussi d'éviter un tas de question sur la forme idéale d'une page listant toutes les exceptions, qui devrait par exemple "merger" les exceptions communes à tous les guichets.

#3

Mis à jour par Frédéric Péters il y a plus de 4 ans

  • Lié à Development #36167: Lever la limite de 600 jours lors de l'import des exceptions ajouté
#4

Mis à jour par Benjamin Dauvergne il y a plus de 4 ans

  • Cacher les exceptions dans le passé (partout)
  • Ne montrer que les 10 prochaines
  • Mettre un lien "Voir plus..." et un tableau paginé qui liste tout (sauf le passé)
#5

Mis à jour par Lauréline Guérin il y a plus de 4 ans

  • Assigné à mis à Lauréline Guérin
#6

Mis à jour par Lauréline Guérin il y a plus de 4 ans

  • Fichier 0002-forms-add-a-query-filter-include-anonymised-to-api-f.patch ajouté
  • Fichier 0001-misc-add-an-utility-function-to-get-flag-from-query-.patch ajouté
  • Statut changé de Nouveau à Solution proposée
  • Patch proposed changé de Non à Oui
#7

Mis à jour par Benjamin Dauvergne il y a plus de 4 ans

  • Statut changé de Solution proposée à Nouveau

Mauvais ticket je pense.

#8

Mis à jour par Lauréline Guérin il y a plus de 4 ans

tout à fait ... :/
voila les bons patchs :)

#9

Mis à jour par Benjamin Dauvergne il y a plus de 4 ans

C'est ok concernant ce que je disais mais en testant le patch en live j'ai quelques remarques :
1. la suppression d'une exception depuis la popup ferme la popup ; je suppose que c'était déjà le cas avant et que ça ne se réglera plutôt coté gadjo et pas facilement maintenant (normalement il a ce qu'il faut pour recharger la popup)
2. la suppression d'une exception depuis le listing paginé ne reste pas sur la page du listing mais revient à la page de settings; il faudrait y rester
3. la popup de confirmation de suppression ne rappelle pas l'intitulé ni les dates de l'exception (c'est pour un autre ticket mais je le signale ici)

Il n'y a que le point 2 qui mérite qu'on s'y attarde sur ce ticket, les autres pourront aller dans deux nouveaux tickets.

#10

Mis à jour par Benjamin Dauvergne il y a plus de 4 ans

  • Statut changé de Solution proposée à Solution validée

Je valide, mais si tu vois comment règler le 2. c'est cool.

#11

Mis à jour par Lauréline Guérin il y a plus de 4 ans

  • Assigné à mis à Lauréline Guérin
#13

Mis à jour par Lauréline Guérin il y a plus de 4 ans

  • Fichier 0001-misc-add-an-utility-function-to-get-flag-from-query-.patch supprimé
#14

Mis à jour par Lauréline Guérin il y a plus de 4 ans

  • Fichier 0002-forms-add-a-query-filter-include-anonymised-to-api-f.patch supprimé
#15

Mis à jour par Lauréline Guérin il y a plus de 4 ans

j'ai ouvert les tickets #37418 et #37419 pour les points 1 et 3

#16

Mis à jour par Benjamin Dauvergne il y a plus de 4 ans

  • Statut changé de Solution proposée à Solution validée
#17

Mis à jour par Lauréline Guérin il y a plus de 4 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit 49eff910c699c1a1d2536dc99da876fcb35d579a
Author: Lauréline Guérin <zebuline@entrouvert.com>
Date:   Mon Nov 4 10:02:41 2019 +0100

    time exceptions: stay on exception list on delete (#19033)

commit c6f418dd8ec4f2b6a77353a701c392e8adaf2806
Author: Lauréline Guérin <zebuline@entrouvert.com>
Date:   Thu Oct 31 14:36:36 2019 +0100

    time exceptions: paginated list of exceptions (#19033)

commit b5ef493d777d0f129ea4cf4afeb45d9ec2e6b16d
Author: Lauréline Guérin <zebuline@entrouvert.com>
Date:   Thu Oct 31 11:59:19 2019 +0100

    time exceptions: never show past exceptions (#19033)
#18

Mis à jour par Frédéric Péters il y a plus de 4 ans

  • Statut changé de Résolu (à déployer) à Solution déployée

Formats disponibles : Atom PDF