Skip to content

Tableau de bord: mise à jour de l'encart du PASS pour les candidats [GEN-2122]#4947

Merged
xavfernandez merged 4 commits intomasterfrom
xfernandez/job_seeker_approval_card
Nov 6, 2024
Merged

Tableau de bord: mise à jour de l'encart du PASS pour les candidats [GEN-2122]#4947
xavfernandez merged 4 commits intomasterfrom
xfernandez/job_seeker_approval_card

Conversation

@xavfernandez
Copy link
Copy Markdown
Contributor

@xavfernandez xavfernandez commented Oct 17, 2024

🤔 Pourquoi ?

Cohérence avec le reste de la refonte.

🍰 Comment ?

En utilisant l'include existant. Comme on a tout de même encore pas mal d'agréments en période de carence, j'ai pris le partie de les afficher a minima en réutilisant le même gabarit.

🚨 À vérifier

  • Mettre à jour le CHANGELOG_breaking_changes.md ?

🏝️ Comment tester

Le plus simple est de se connecter avec:

  • le compte de demo candidat d'un coté
  • et avec l'admin de l'autre

et tester les différents cas:

  • Candidat sans agrément (en modifiant l'identifiant FT par exemple)
  • Candidat en délai de carence ou pas en modifiant la date de fin de l'agrément
  • Candidat avec diag valid ou non en lui attribuant/supprimant un diag valide
  • etc

💻 Captures d'écran

Screenshot 2024-11-05 at 16-02-43 Tableau de bord - Les emplois de l'inclusion
Screenshot 2024-11-05 at 16-02-32 Tableau de bord - Les emplois de l'inclusion

@xavfernandez xavfernandez added the modifié Modifié dans le changelog. label Oct 17, 2024
@notion-workspace
Copy link
Copy Markdown

@xavfernandez xavfernandez self-assigned this Oct 17, 2024
@xavfernandez xavfernandez added the 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC label Oct 17, 2024
@github-actions
Copy link
Copy Markdown

🥁 La recette jetable est prête ! 👉 Je veux tester cette PR !

@xavfernandez xavfernandez force-pushed the xfernandez/job_seeker_approval_card branch 7 times, most recently from 24f3412 to a0f8c8c Compare October 18, 2024 12:53
@xavfernandez xavfernandez marked this pull request as ready for review October 18, 2024 12:53
Comment on lines +81 to +102
{% if job_seeker_help_text|default:"" == "waiting_period_with_valid_diagnosis" %}
<hr class="my-3 my-md-4">
<p class="fs-sm mb-0">
Un prescripteur habilité a réalisé un diagnostic d'éligibilité. <strong>Vous pouvez commencer un nouveau parcours.</strong>
</p>
{% elif job_seeker_help_text|default:"" == "waiting_period_without_diagnosis" %}
<hr class="my-3 my-md-4">
<p class="fs-sm mb-0">Vous avez terminé un parcours il y a moins de deux ans.</p>
<p class="fs-sm my-0">
Pour prétendre à nouveau à un parcours en structure d'insertion par l'activité économique vous devez rencontrer un prescripteur
habilité : France Travail, Mission Locale, Cap emploi, etc.
</p>
{% endif %}
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

J'hésite avec:

Suggested change
{% if job_seeker_help_text|default:"" == "waiting_period_with_valid_diagnosis" %}
<hr class="my-3 my-md-4">
<p class="fs-sm mb-0">
Un prescripteur habilité a réalisé un diagnostic d'éligibilité. <strong>Vous pouvez commencer un nouveau parcours.</strong>
</p>
{% elif job_seeker_help_text|default:"" == "waiting_period_without_diagnosis" %}
<hr class="my-3 my-md-4">
<p class="fs-sm mb-0">Vous avez terminé un parcours il y a moins de deux ans.</p>
<p class="fs-sm my-0">
Pour prétendre à nouveau à un parcours en structure d'insertion par l'activité économique vous devez rencontrer un prescripteur
habilité : France Travail, Mission Locale, Cap emploi, etc.
</p>
{% endif %}
{% if extra_text|default:"" %}
<hr class="my-3 my-md-4">
{{ extra_text }}
{% endif %}

et laisser la vue injecter son mark_safe/format_html en variable.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Je pense que je préfère le 1er.

Le second a l'avantage de permettre de mettre ce qu'on veut (si on veut 15 versions possibles) mais

  • je ne pense pas qu'on en aura vraiment beaucoup plus.
  • il faut mettre les balises html dans le code python (ce que je préfère éviter quand je peux)
  • ou alors passer un extra_texts et faire {% for extra_text in extras_texts %} mais c'est encore plus compliqué pour pas grand chose

@xavfernandez xavfernandez force-pushed the xfernandez/job_seeker_approval_card branch from a0f8c8c to 51283c9 Compare October 18, 2024 13:02
Copy link
Copy Markdown
Contributor

@tonial tonial left a comment

Choose a reason for hiding this comment

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

👍

Comment on lines +81 to +102
{% if job_seeker_help_text|default:"" == "waiting_period_with_valid_diagnosis" %}
<hr class="my-3 my-md-4">
<p class="fs-sm mb-0">
Un prescripteur habilité a réalisé un diagnostic d'éligibilité. <strong>Vous pouvez commencer un nouveau parcours.</strong>
</p>
{% elif job_seeker_help_text|default:"" == "waiting_period_without_diagnosis" %}
<hr class="my-3 my-md-4">
<p class="fs-sm mb-0">Vous avez terminé un parcours il y a moins de deux ans.</p>
<p class="fs-sm my-0">
Pour prétendre à nouveau à un parcours en structure d'insertion par l'activité économique vous devez rencontrer un prescripteur
habilité : France Travail, Mission Locale, Cap emploi, etc.
</p>
{% endif %}
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Je pense que je préfère le 1er.

Le second a l'avantage de permettre de mettre ce qu'on veut (si on veut 15 versions possibles) mais

  • je ne pense pas qu'on en aura vraiment beaucoup plus.
  • il faut mettre les balises html dans le code python (ce que je préfère éviter quand je peux)
  • ou alors passer un extra_texts et faire {% for extra_text in extras_texts %} mais c'est encore plus compliqué pour pas grand chose

Comment thread itou/www/dashboard/views.py Outdated
@xavfernandez xavfernandez force-pushed the xfernandez/job_seeker_approval_card branch 4 times, most recently from e13f4a3 to c98c7e2 Compare November 5, 2024 08:27
Copy link
Copy Markdown
Member

@francoisfreitag francoisfreitag left a comment

Choose a reason for hiding this comment

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

Autrement LGTM :)

Comment thread itou/templates/approvals/includes/box.html Outdated
Comment thread tests/www/approvals_views/test_detail.py
@xavfernandez xavfernandez force-pushed the xfernandez/job_seeker_approval_card branch 3 times, most recently from cef84b5 to 255c760 Compare November 6, 2024 09:01
@xavfernandez xavfernandez force-pushed the xfernandez/job_seeker_approval_card branch 2 times, most recently from e988f3c to 8a2d0c5 Compare November 6, 2024 10:43
@xavfernandez xavfernandez force-pushed the xfernandez/job_seeker_approval_card branch from 8a2d0c5 to 38af4e9 Compare November 6, 2024 11:01
Comment thread itou/templates/dashboard/includes/job_seeker_approval_card.html Outdated
@xavfernandez xavfernandez force-pushed the xfernandez/job_seeker_approval_card branch from 38af4e9 to 9f977b0 Compare November 6, 2024 11:53
@xavfernandez xavfernandez added this pull request to the merge queue Nov 6, 2024
Merged via the queue into master with commit 623e6ae Nov 6, 2024
@xavfernandez xavfernandez deleted the xfernandez/job_seeker_approval_card branch November 6, 2024 12:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC modifié Modifié dans le changelog.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants