Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[CW-512] Implémenter la récupération des user_dict dans la liste des jobs retournée #193

Merged

Conversation

notoraptor
Copy link
Contributor

Bonjour! Voici une PR pour s'assurer que les user props sont récupérées avec les jobs retournés par get_jobs().

Comme expliqué, cela avait déjà été implémenté dans la PR #185 . Les principales lignes de code concernées sont visibles ici (get_jobs() appelle get_filtered_and_paginated_jobs()): https://github.com/mila-iqia/clockwork/blob/master/clockwork_web/core/jobs_helper.py#L161

Cette PR-ci ajoute seulement un test pour vérifier que les user props sont bel et bien retournées avec les jobs.

PS:

Pour implémenter la PR 185, je m'étais basé sur mon ancien code (désormais obsolète) relatif au regroupement des jobs par label. Du coup, j'avais conservé la fonctionnalité de regroupement: c'est-à-dire que, si on clique sur une user prop dans le tableau de la page des jobs, alors ça va filtrer uniquement tous les jobs qui ont cette user-prop là (tous les jobs qui auront cette clé associée à cette valeur).

Mais je me demande si on voulait avoir ce comportement-là pour les user props, aussi. Si non, je pourrais profiter de cette PR pour le retirer.

Exemple ici:

Capture.d.ecran.video.de.2024-05-31.08.19.11.AM.webm

@soline-b @gyom

@notoraptor notoraptor changed the title Add a test to check that get_jobs() returns user props. [CW-512] Implémenter la récupération des user_dict dans la liste des jobs retournée May 31, 2024
@soline-b soline-b merged commit 8d9266e into mila-iqia:master Jun 12, 2024
2 checks passed
@notoraptor notoraptor deleted the cw-512-get-user-props-with-jobs branch June 13, 2024 13:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants