Development #23298
Lien vers une ligne de log précise
0%
Description
De Thomas dans #14671 :
Ca me plairait que sur chaque ligne affichée dans resource-logs-table.html, on fasse un lien vers « view-logs-connector?=q=datetime_de_la_ligne », par exemple au niveau de la colonne timestamp. Comme ça hop, ça nous fait une URL partageable avec les collègues très facilement.
Fichiers
Révisions associées
Historique
Mis à jour par Thomas Noël il y a environ 6 ans
- Fichier 0001-manager-add-link-to-full-page-logs-on-each-log-line-.patch 0001-manager-add-link-to-full-page-logs-on-each-log-line-.patch ajouté
- Statut changé de Nouveau à En cours
- Patch proposed changé de Non à Oui
Mis à jour par Frédéric Péters il y a environ 6 ans
Je ne sais pas pour le changement à l'affichage, {{ record.timestamp|localtime|date:"Y-m-d H:i:s" }}; avoir la date/heure en français j'aime bien.
Mis à jour par Thomas Noël il y a environ 6 ans
Et donc justement non, pas le truc par défaut, parce que je voulais les secondes. Je vais mixer, date localisée, mais heure "internationale (sur 24 heures)"
Mis à jour par Thomas Noël il y a environ 6 ans
Mis à jour par Benjamin Dauvergne il y a environ 6 ans
En lien avec #23312 le lien pourrait être aussi ?after=<record-timestamp-isoformat>,<record.pk>, ça ramène la ligne exacte.
Mis à jour par Lauréline Guérin il y a environ 4 ans
- Fichier 0001-logs-add-a-link-to-a-search-per-date-23298.patch 0001-logs-add-a-link-to-a-search-per-date-23298.patch ajouté
- Statut changé de En cours à Solution proposée
Si on reste sur l'idée: lancer une recherche sur la date avec heure exacte, alors le patch d'il y a 2 ans répond à la question (patch soumis en pièce jointe)
Si on est plutôt sur: arriver sur la bonne page, alors il faut revoir le système de pagination, et passer sur de la pagination par clé (ce qui serait une bonne idée du point de vue perfs)
On peut toujours résoudre ce ticket simplement (juste une recherche sur la date avec heure exacte) et revoir le système de pagination dans un autre ticket.
Mis à jour par Valentin Deniaud il y a environ 4 ans
Il y a #38157 qui couvre sûrement déjà au cas d'usage « partager une ligne de log », aussi.
Mis à jour par Lauréline Guérin il y a environ 4 ans
Fred, Thomas est-ce que #38157 répond aussi à ce ticket ?
Mis à jour par Thomas Noël il y a environ 4 ans
Lauréline Guerin a écrit :
Fred, Thomas est-ce que #38157 répond aussi à ce ticket ?
De ce que je comprends, #38157 va renvoyer une série de lignes (concernant un échange donné), c'est fort utile. Ici on aimerait pouvoir pointer une ligne en particulier, effectivement je pense que c'est pas vraiment nécessaire si on a déjà #38157. Vous vouliez mon avis vous l'avez :)
Mis à jour par Frédéric Péters il y a environ 4 ans
Je dirais que pas tout à fait, ça va permettre de pointer une série de lignes mais pas une ligne précise. Mais en fait j'ai une interprétation du ticket un peu différente de la partie citée dans la description, je voudrais pouvoir pointer une ligne précise; précisément j'aimerais pouvoir partager un lien qui affiche le tableau des logs, avec une seule ligne présente, qu'il soit derrière possible de cliquer dessus pour avoir les détails, voir les autres appels en rapport (#38157), etc.
Aujourd'hui je peux regarder les requêtes qui passent à l'ouverture de la popup et filer une URL comme
https://passerelle-strasbourg.test.entrouvert.org/manage/base-adresse/geocodage-via-adict/logs/5156445/
J'aimerais plutôt de la popup avoir la possibilité d'obtenir un lien, https://passerelle-strasbourg.test.entrouvert.org/manage/base-adresse/geocodage-via-adict/logs/?log_id=5156445, qui filtre le tableau sur cette seule ligne.
En fait à suivre une relecture qui dirait dans #38157 que le lien "search for logs from the same call" devrait être posé différemment (faudrait vraiment que je fasse la relecture), on pourrait imaginer que la popup d'un log soit :
propriété 1 .... propriété 2 .... [Afficher tous les logs de l'appel] [Lien vers cette fenêtre]
Et [Lien vers cette fenêtre] serait l'objet de ce ticket, et contiendrait donc
et on peut imaginer alors qu'un appel à la page avec ?log_id=... ouvre automatiquement la popup de la ligne en question.
(pas fan du tout de mes libellés de boutons pour la boite de dialogue)
Mis à jour par Lauréline Guérin il y a environ 4 ans
- Statut changé de Solution proposée à En cours
Je vais attendre que le #38157 soit passé pour reprendre ce ticket
Mis à jour par Lauréline Guérin il y a environ 4 ans
- Fichier 0001-logs-add-a-link-to-a-search-per-date-23298.patch 0001-logs-add-a-link-to-a-search-per-date-23298.patch ajouté
- Statut changé de En cours à Solution proposée
Mis à jour par Frédéric Péters il y a environ 4 ans
- Statut changé de Solution proposée à Solution validée
Ça marche bien et correspond tout à fait à ce que j'écrivais plus haut, évidemment maintenant je me demande s'il y a moyen de jouer en plus avec window.history.pushState (https://developer.mozilla.org/fr/docs/Web/API/History/pushState), que l'adresse affichée gagne/perde ?log_id=... à l'ouverture/fermeture d'une popup, et comme ça même pas avoir besoin du bouton dans la fenêtre.
Si ça te dit de regarder, mais tu peux très bien pousser ainsi.
Mis à jour par Lauréline Guérin il y a environ 4 ans
- Fichier 0001-logs-direct-access-to-a-log-23298.patch 0001-logs-direct-access-to-a-log-23298.patch ajouté
- Statut changé de Solution validée à Solution proposée
Voila, avec manipulation de l'historique.
C'est pas mal, ça permet même de s'affranchir d'une interaction malvenue entre log_id et la pagination
Mis à jour par Frédéric Péters il y a environ 4 ans
- Statut changé de Solution proposée à Solution validée
C'est super, go.
Mis à jour par Lauréline Guérin il y a environ 4 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit f72841b9fc7248149e61fc135838cdd68c011584
Author: Lauréline Guérin <zebuline@entrouvert.com>
Date: Fri Mar 20 14:28:22 2020 +0100
logs: direct access to a log (#23298)
Mis à jour par Frédéric Péters il y a environ 4 ans
- Statut changé de Résolu (à déployer) à Solution déployée
logs: direct access to a log (#23298)