entrouvert
/
passerelle
15
0
Fork 0
Code Issues Pull Requests 16 Releases Activity

Pouvoir filtrer les jobs sur leur statut dans la vue des jobs (#88641) #503

Open
bdauvergne wants to merge 3 commits from wip/88641-jobs-pouvoir-filtrer-sur-le-stat into main
pull from: wip/88641-jobs-pouvoir-filtrer-sur-le-stat
merge into: entrouvert:main
Conversation 4 Commits 3 Files Changed 5 +123 -21
bdauvergne commented 2024-03-28 16:20:47 +01:00
Owner
Copy Link
No description provided.
bdauvergne added 3 commits 2024-03-28 16:20:47 +01:00
gitea/passerelle/pipeline/head This commit looks good Details
492ecec648 misc: add filter on job status to job page (#88641)
bdauvergne force-pushed wip/88641-jobs-pouvoir-filtrer-sur-le-stat from 492ecec648 to ecfb9e50f9 2024-03-28 17:07:40 +01:00 Compare
bdauvergne changed title from WIP: Pouvoir filtrer les jobs sur leur statut dans la vue des jobs (#88641) to Pouvoir filtrer les jobs sur leur statut dans la vue des jobs (#88641) 2024-03-28 17:12:36 +01:00
nroche requested changes 2024-04-15 11:47:20 +02:00
Dismissed
nroche left a comment
Owner
Copy Link

Le fait que lorsqu'on arrive sur la page /jobs sans passer la paramètre job-status-filter n'affiche rien,
fait que l'on perd la possibilité de rechercher sur la date ou le nom de la tâche (la recherche n'affiche plus de résultats).

Le fait que lorsqu'on arrive sur la page /jobs sans passer la paramètre job-status-filter n'affiche rien, fait que l'on perd la possibilité de rechercher sur la date ou le nom de la tâche (la recherche n'affiche plus de résultats).
passerelle/templates/passerelle/manage/service_jobs.html Outdated
@ -16,0 +19,4 @@
<span class="job-status">
{% if job_status_filter != slug %}<a href="?job-status-filter={{ slug }}{% if query %}&q={{ query }}{% endif %}">{% endif %}
{{ label }} ({{ count }})
{% if status != slug %}</a>{% endif %}
nroche commented 2024-04-15 11:35:31 +02:00
Owner
Copy Link

Typo : plutôt job_status_filter, je pense.

Typo : plutôt job_status_filter, je pense.
bdauvergne commented 2024-04-15 15:37:33 +02:00
Author
Owner
Copy Link

C'est corrigé.

C'est corrigé.
bdauvergne commented 2024-04-15 15:34:53 +02:00
Author
Owner
Copy Link

Le fait que lorsqu'on arrive sur la page /jobs sans passer la paramètre job-status-filter n'affiche rien,
fait que l'on perd la possibilité de rechercher sur la date ou le nom de la tâche (la recherche n'affiche plus de résultats).

Oui c'est filtré par défaut sur tous les statuts sauf terminé, c'est visible dans la barre de filtre, si tu n'as pas de job en erreur ou en cours c'est vide effectivement.

> Le fait que lorsqu'on arrive sur la page /jobs sans passer la paramètre job-status-filter n'affiche rien, > fait que l'on perd la possibilité de rechercher sur la date ou le nom de la tâche (la recherche n'affiche plus de résultats). Oui c'est filtré par défaut sur tous les statuts sauf terminé, c'est visible dans la barre de filtre, si tu n'as pas de job en erreur ou en cours c'est vide effectivement.
bdauvergne force-pushed wip/88641-jobs-pouvoir-filtrer-sur-le-stat from ecfb9e50f9 to 87d8b23222 2024-04-15 15:37:24 +02:00 Compare
bdauvergne requested review from nroche 2024-04-15 15:37:36 +02:00
nroche commented 2024-04-15 17:28:52 +02:00
Owner
Copy Link

Le fait que lorsqu'on arrive sur la page /jobs sans passer la paramètre job-status-filter n'affiche rien,

Oui c'est filtré par défaut sur tous les statuts sauf terminé

Compris merci.
Pour moi c'est assez contre-intuitif que la recherche ne remonte pas les jobs terminés, pourtant visibles en passant par les critères.
Peut-être préciser l'aide : '(recherche des jobs non terminés sur la date ou le nom de la tâche)' ?

> > Le fait que lorsqu'on arrive sur la page /jobs sans passer la paramètre job-status-filter n'affiche rien, > Oui c'est filtré par défaut sur tous les statuts sauf terminé Compris merci. Pour moi c'est assez contre-intuitif que la recherche ne remonte pas les jobs terminés, pourtant visibles en passant par les critères. Peut-être préciser l'aide : '(recherche des jobs non terminés sur la date ou le nom de la tâche)' ?
nroche approved these changes 2024-04-15 17:30:35 +02:00
nroche left a comment
Owner
Copy Link

Ack.
Comme je le disais dans mon commentaire :
Peut-être en profiter pour préciser l'aide sur la recherche : '(recherche des jobs non terminés sur la date ou le nom de la tâche)' ?

Ack. Comme je le disais dans mon commentaire : Peut-être en profiter pour préciser l'aide sur la recherche : '(recherche des jobs non terminés sur la date ou le nom de la tâche)' ?
bdauvergne commented 2024-04-15 18:28:50 +02:00
Author
Owner
Copy Link

Ack.
Comme je le disais dans mon commentaire :
Peut-être en profiter pour préciser l'aide sur la recherche : '(recherche des jobs non terminés sur la date ou le nom de la tâche)' ?

Tu veux que je rajoute un choix "Tous" après "Tous sauf terminés" ? Mon intuition c'est de maximiser le cas de la recherche d'un souci ce qu'a tenté de faire Cyril mais n'a pas pu les erreurs étant noyées dans la masse. La recherche d'un job qui a bien fonctionné ça ne me parait pas le cas d'usage le plus utile (mais ça reste possible) même si bien sûr on le fait pendant une phase de debug.

> Ack. > Comme je le disais dans mon commentaire : > Peut-être en profiter pour préciser l'aide sur la recherche : '(recherche des jobs non terminés sur la date ou le nom de la tâche)' ? Tu veux que je rajoute un choix "Tous" après "Tous sauf terminés" ? Mon intuition c'est de maximiser le cas de la recherche d'un souci ce qu'a tenté de faire Cyril mais n'a pas pu les erreurs étant noyées dans la masse. La recherche d'un job qui a bien fonctionné ça ne me parait pas le cas d'usage le plus utile (mais ça reste possible) même si bien sûr on le fait pendant une phase de debug.
bdauvergne requested review from nroche 2024-04-15 18:57:45 +02:00
nroche commented 2024-04-17 10:51:41 +02:00
Owner
Copy Link

Je sais pas bien au juste (au pire j'ai déjà validé le ticket).

Dans le ticket quand tu dis "par défaut ne pas afficher les tâches terminées",
ça concerne aussi la recherche ?

Je note ici que ce patch limite les résultats de la recherches, par rapport au comportement que l'on a actuellement.
Aussi, je réalise que l'onglet dans "Tâches asynchrones" de la page du connecteur affiche les jobs terminés alors que la page avancée ne les affiche plus.

Tu veux que je rajoute un choix "Tous" après "Tous sauf terminés" ?

J'ai l'impression que si tu ajoutes "Tous" AVANT "Tous sauf terminés",
alors le patch n'introduira pas de modification sur le comportement existant sur la recherche.
ie : que le choix "Tous sauf terminés", qui bien qu'il soit super pratique, ne soit pas le mode par défaut.

else:
            return qs#.exclude(status='completed')

La recherche d'un job qui a bien fonctionné ça ne me parait pas le cas d'usage le plus utile (mais ça reste possible) même si bien sûr on le fait pendant une phase de debug.

Je réalise (seulement maintenant) qu'avec ce patch, quand on précise une recherche, on peut ensuite la filtrer.
Je trouve dommage que l'on ne puisse pas, à l'inverse, filtrer puis lancer une recherche.
A toute fin utile, je conseillerais soit d'inverser la présentation : afficher la recherche au dessus des filtres,
soit de conserver le filtre quand on lance la recherche ;
ie : ajouter un champ hiden avec le status sur le formulaire de recherche (dans passerelle/templates/passerelle/manage/service_jobs.html) :
<input type="hidden" name="job-status-filter" value="{{ job_status_filter }}">

Je sais pas bien au juste (au pire j'ai déjà validé le ticket). Dans le ticket quand tu dis "par défaut ne pas afficher les tâches terminées", ça concerne aussi la recherche ? Je note ici que ce patch limite les résultats de la recherches, par rapport au comportement que l'on a actuellement. Aussi, je réalise que l'onglet dans "Tâches asynchrones" de la page du connecteur affiche les jobs terminés alors que la page avancée ne les affiche plus. > Tu veux que je rajoute un choix "Tous" après "Tous sauf terminés" ? J'ai l'impression que si tu ajoutes "Tous" AVANT "Tous sauf terminés", alors le patch n'introduira pas de modification sur le comportement existant sur la recherche. ie : que le choix "Tous sauf terminés", qui bien qu'il soit super pratique, ne soit pas le mode par défaut. ``` else: return qs#.exclude(status='completed') ``` > La recherche d'un job qui a bien fonctionné ça ne me parait pas le cas d'usage le plus utile (mais ça reste possible) même si bien sûr on le fait pendant une phase de debug. Je réalise (seulement maintenant) qu'avec ce patch, quand on précise une recherche, on peut ensuite la filtrer. Je trouve dommage que l'on ne puisse pas, à l'inverse, filtrer puis lancer une recherche. A toute fin utile, je conseillerais soit d'inverser la présentation : afficher la recherche au dessus des filtres, soit de conserver le filtre quand on lance la recherche ; ie : ajouter un champ hiden avec le status sur le formulaire de recherche (dans passerelle/templates/passerelle/manage/service_jobs.html) : ```<input type="hidden" name="job-status-filter" value="{{ job_status_filter }}">```
All checks were successful
gitea/passerelle/pipeline/head This commit looks good
Details
This pull request can be merged automatically.
This branch is out-of-date with the base branch
You are not authorized to merge this pull request.
Sign in to join this conversation.
Reviewers
No reviewers
nroche
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/passerelle#503
No description provided.
Delete Branch "wip/88641-jobs-pouvoir-filtrer-sur-le-stat"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?