Skip to content

Admin: Amélioration de la page des candidature pour les actions du support#4379

Merged
tonial merged 4 commits intomasterfrom
alaurent/job_app_admin
Jul 8, 2024
Merged

Admin: Amélioration de la page des candidature pour les actions du support#4379
tonial merged 4 commits intomasterfrom
alaurent/job_app_admin

Conversation

@tonial
Copy link
Contributor

@tonial tonial commented Jul 5, 2024 •
edited
Loading

🤔 Pourquoi ?

Suite aux changement de status, voir : https://itou-inclusion.slack.com/archives/C02J7LWNT6Z/p1720176613306289?thread_ts=1718717668.744509&cid=C02J7LWNT6Z

🍰 Comment ?

  • Le bouton "Annuler" est à présent toujours visible sur une candidature acceptée, mais grisé quand il y a des fiches salariées qui bloquent l'annulation, avec un tooltip.
  • Les fiches salariées d'une candidature sont visible depuis la page de celle-ci pour les retrouver plus rapidement quand elles doivent être supprimées pour régulation.
  • On peut accepter depuis l'admin un candidature avec un vieil agrément (on fait confiance au support)
  • On vérifie que le diagnostic appartient au candidat

🚨 À vérifier

  • Mettre à jour le CHANGELOG_breaking_changes.md ?

🏝️ Comment tester

Les instructions pour reproduire le problème, les profils de test, le parcours spécifique à utiliser, etc. Si vous disposez d'une recette jetable, mettre l'URL pour tester dans cette partie.

💻 Captures d'écran

Le bouton accepter avec le commentaire :
image

@tonial tonial added the modifié Modifié dans le changelog. label Jul 5, 2024
@tonial tonial requested a review from xavfernandez July 5, 2024 13:14
@tonial tonial self-assigned this Jul 5, 2024
xavfernandez
xavfernandez reviewed Jul 5, 2024
View reviewed changes
Copy link
Contributor

@xavfernandez xavfernandez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Il faudrait idéalement découper en 3(?) commits pour expliquer les différentes améliorations/modifications.

xavfernandez
xavfernandez approved these changes Jul 5, 2024
View reviewed changes
itou/job_applications/models.py Outdated
Comment on lines 1008 to 1017
# If request user is itou_staff keep the existing eligibility diagnosis
if user.kind != UserKind.ITOU_STAFF or self.eligibility_diagnosis is None:
Copy link
Contributor

@xavfernandez xavfernandez Jul 5, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Je me demande si on ne voudrait pas un message dans l'admin pour les canidatures pouvant être acceptées et ayant un diag expiré pour dire qu'on ne vérifiera pas la validité du diag sélectionné (et qu'il faut donc faire gaffe ou le supprimer si on veut que le système prenne le dernier) ?

Copy link
Contributor Author

@tonial tonial Jul 5, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tu le mettrais où ce message ?

Copy link
Contributor

@xavfernandez xavfernandez Jul 5, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Peut-être dans l'encart des boutons d'action à coté du bouton Accepter s'il est dispo ?

@tonial tonial force-pushed the alaurent/job_app_admin branch 2 times, most recently from 6e4abe3 to 28f4a2e Compare July 5, 2024 14:03
@tonial
Copy link
Contributor Author

tonial commented Jul 5, 2024

@xavfernandez j'ai ajouté un check sur le candidat du diag dans l'admin form et découpé en commits distincts :)

@tonial tonial force-pushed the alaurent/job_app_admin branch from 28f4a2e to 67517b3 Compare July 5, 2024 14:04
xavfernandez
xavfernandez approved these changes Jul 5, 2024
View reviewed changes
itou/job_applications/models.py Outdated
Comment on lines 1008 to 1017
# If request user is itou_staff keep the existing eligibility diagnosis
if user.kind != UserKind.ITOU_STAFF or self.eligibility_diagnosis is None:
Copy link
Contributor

@xavfernandez xavfernandez Jul 5, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Peut-être dans l'encart des boutons d'action à coté du bouton Accepter s'il est dispo ?

@tonial tonial force-pushed the alaurent/job_app_admin branch 4 times, most recently from f35261b to 13f042d Compare July 5, 2024 14:28
@tonial tonial force-pushed the alaurent/job_app_admin branch from 13f042d to 2b67a27 Compare July 8, 2024 06:45
@tonial tonial enabled auto-merge July 8, 2024 06:45
@tonial tonial force-pushed the alaurent/job_app_admin branch from 2b67a27 to 464b3be Compare July 8, 2024 07:30
@tonial tonial added this pull request to the merge queue Jul 8, 2024
Merged via the queue into master with commit 7efbcfb Jul 8, 2024
@tonial tonial deleted the alaurent/job_app_admin branch July 8, 2024 07:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@xavfernandez xavfernandez xavfernandez approved these changes

Assignees

@tonial tonial

Labels

modifié Modifié dans le changelog.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@tonial @xavfernandez