Prescripteur : ajout d'une page "Présentation" pour l'organisation des prescripteurs habilités et d'une entrée dans le menu latéral by hellodeloo · Pull Request #6060 · gip-inclusion/les-emplois

hellodeloo · 2025-04-28T15:40:04Z

🤔 Pourquoi ?

Tous comme les employeurs, les prescripteurs et les orienteurs doivent avoir accès à une page de présentation de leur organisation pour avoir une vue d’ensemble et accéder à certaines actions qui vont disparaitre de l’accueil

notion-workspace · 2025-04-28T15:40:17Z

🌿 ETQ prescripteur habilité / orienteur, j’ai accès à une page de présentation de mon organisation

francoisfreitag

En dehors du souci de permission, la page me semble bien fonctionner et être présente dans les menus comme souhaité 👍

Je ne vois pas le bouton « Voir la fiche publique » de la maquette ?

EwenKorr · 2025-04-29T14:00:48Z

Je ne vois pas le bouton « Voir la fiche publique » de la maquette ?

Il est actuellement conditionné à l'habilitation de l'organisation (il n'y a que les prescripteurs habilités qu'on peut trouver dans la recherche prescripteur).

Comme ça par exemple :

diff --git a/itou/prescribers/models.py b/itou/prescribers/models.py
index f09dc9c90..b693242e2 100644
--- a/itou/prescribers/models.py
+++ b/itou/prescribers/models.py
@@ -270,8 +270,6 @@ class PrescriberOrganization(AddressMixin, OrganizationAbstract):
         return get_tally_form_url("w2EoDp", **kwargs)

     def get_card_url(self):
-        if not self.is_authorized:
-            return None
         return reverse("prescribers_views:card", kwargs={"org_id": self.pk})

     def has_refused_authorization(self):
diff --git a/itou/templates/prescribers/card.html b/itou/templates/prescribers/card.html
index 8703bb384..8b1bbb5e1 100644
--- a/itou/templates/prescribers/card.html
+++ b/itou/templates/prescribers/card.html
@@ -13,7 +13,7 @@
 {% block title_content %}
     {% component_title c_title__main=c_title__main c_title__secondary=c_title__secondary %}
         {% fragment as c_title__main %}
-            <h1>Prescripteur habilité</h1>
+           <h1>Prescripteur{% if prescriber_org.is_authorized %}  habilité{% endif %}</h1>
         {% endfragment %}
         {% fragment as c_title__secondary %}
             <h2>{{ prescriber_org.get_kind_display }} - {{ prescriber_org.name }}</h2>
diff --git a/itou/www/prescribers_views/views.py b/itou/www/prescribers_views/views.py
index de1b405c3..f6d3bc55b 100644
--- a/itou/www/prescribers_views/views.py
+++ b/itou/www/prescribers_views/views.py
@@ -22,7 +22,6 @@ def card(request, org_id, template_name="prescribers/card.html"):
     prescriber_org = get_object_or_404(
         PrescriberOrganization,
         pk=org_id,
-        authorization_status=PrescriberAuthorizationStatus.VALIDATED,
     )
     back_url = get_safe_url(request, "back_url")
     context = {

Tu aurais besoin d'aide pour les tests de permission David ?

francoisfreitag · 2025-04-29T14:46:21Z

Je ne vois pas le bouton « Voir la fiche publique » de la maquette ?

Il est actuellement conditionné à l'habilitation de l'organisation (il n'y a que les prescripteurs habilités qu'on peut trouver dans la recherche prescripteur).

Bien vu, j’aurais du chercher la chaîne de caractères dans le HTML au lieu de me fier à mes yeux haha.

github-actions · 2025-04-30T10:14:05Z

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

EwenKorr · 2025-05-02T13:13:18Z

J'ai rajouté des permissions et des tests.
Il faudra voir les commits qu'on squashe.

EwenKorr · 2025-05-05T13:56:04Z

+                                {% endif %}
+                            </div>
+                            <div class="col-12 col-lg-4 order-1 order-lg-2">
+                                {% include "companies/includes/_company_info.html" with company=organization show=True show_cta=False extra_box_class="mb-3 mb-lg-5" %}


On inclut ici un bout d'HTML de companies.
Mon cœur balance entre séparer les entreprises des prescripteurs, et le principe DRY.
Un avis ?

Peut-être un renommage pour déplacer ce fragment vers un emplacement plus générique ?

Je propose toujours un déplacement ici, pour mieux indiquer que le fragment fonctionne aussi avec les organisations.

Ah oui !
Je ferai ça dans une PR à côté, je me note ça.

francoisfreitag

En me connectant en tant qu’orienteur membre d’une orga, j’ai dans la console:

Uncaught TypeError: can't access property "focus", thisElToFocus is null
    focusAndScrollTo http://localhost:8000/prescribers/overview:744
    <anonymous> http://localhost:8000/prescribers/overview:750

Le texte de présentation ne s’affiche également pas chez moi (ni chromium, ni firefox)

francoisfreitag · 2025-05-05T14:54:02Z

+                                {% endif %}
+                            </div>
+                            <div class="col-12 col-lg-4 order-1 order-lg-2">
+                                {% include "companies/includes/_company_info.html" with company=organization show=True show_cta=False extra_box_class="mb-3 mb-lg-5" %}


Peut-être un renommage pour déplacer ce fragment vers un emplacement plus générique ?

EwenKorr · 2025-05-05T15:14:42Z

En me connectant en tant qu’orienteur membre d’une orga, j’ai dans la console:

Uncaught TypeError: can't access property "focus", thisElToFocus is null
    focusAndScrollTo http://localhost:8000/prescribers/overview:744
    <anonymous> http://localhost:8000/prescribers/overview:750

C'est le cas partout, y compris actuellement en production ? @hellodeloo, une idée ?

Le texte de présentation ne s’affiche également pas chez moi (ni chromium, ni firefox)

C'est voulu 🤷
https://gip-inclusion.slack.com/archives/CU8ATF54L/p1746438370497209?thread_ts=1746008376.030279&cid=CU8ATF54L

francoisfreitag

Idéalement, prescribers: better display organization kind for France Travail pourrait sortir dans une autre PR plutôt que de rester dans la PR intitulée « ajout d'une page "presentation" pour l'organisation des prescripteurs habilités »

francoisfreitag · 2025-05-12T10:05:01Z

                            {% bootstrap_field form.description %}

-                            {% itou_buttons_form primary_label="Enregistrer" primary_aria_label="Enregistrer la modification" %}
+                            {% if organization.kind != PrescriberOrganizationKind.FT %}


On pourrait centraliser cette logique dans le formulaire, via un attribut (nommé readonly ?). On a déjà

les-emplois/itou/www/prescribers_views/forms.py

Line 56 in 997c499

if self.instance.kind == PrescriberOrganizationKind.FT:

L’attribut aurait idéalement un petit commentaire qui explique pourquoi FT est traité en lecture seule (les collaborateurs changeant d’agence éditaient souvent leur ancienne organisation avec les infos de leur nouvelle agence, faisant un beau micmac), et on aurait ainsi un lien entre les champs désactivés et les boutons masqués.

Bonne idée, de lier la désactivation des champs et le masquage des boutons, je fais ça.

Pour le commentaire : il est déjà là non ?

les-emplois/itou/www/prescribers_views/forms.py

Lines 53 to 55 in 68e082d

# PE users often mistakenly edit this page, to the point where the

# support asked to disable it, and have them reach out in case of

# changes.

(et j'imagine que ce commit aurait mieux été dans une PR séparée aussi 😬)

Le commentaire est effectivement déjà là 🙈.

(oui, le commit aurait aussi été mieux dans une PR séparée 🙂. Après, on est quand même proche de la fin sur celle ci, donc comme tu préfères.)

Je vais laisser ce commit là-dedans et je tâcherai de songer plus tôt à extraire des commits par la suite (notamment à la reprise d'une PR de quelqu'un d'autre :D)

EwenKorr · 2025-05-12T13:10:32Z

Idéalement, prescribers: better display organization kind for France Travail pourrait sortir dans une autre PR plutôt que de rester dans la PR intitulée « ajout d'une page "presentation" pour l'organisation des prescripteurs habilités »

Oui, c'est sûr !
#6110
(ça permet de découper encore mieux, il y avait une amélioration de @hellodeloo sur le composant à intégrer aussi, un <h2> à passer en <p> en gros)

francoisfreitag · 2025-05-12T14:08:47Z

+                                {% endif %}
+                            </div>
+                            <div class="col-12 col-lg-4 order-1 order-lg-2">
+                                {% include "companies/includes/_company_info.html" with company=organization show=True show_cta=False extra_box_class="mb-3 mb-lg-5" %}


Je propose toujours un déplacement ici, pour mieux indiquer que le fragment fonctionne aussi avec les organisations.

EwenKorr · 2025-05-15T12:12:02Z

Dernière petite modif : pour les prescripteurs non habilités, au lieu d'afficher dans le menu latéral seulement "Collaborateurs", on affiche "Modifier les informations" (pour que ça corresponde au bloc du dashboard)

francoisfreitag · 2025-05-15T12:35:16Z

Je ne vois pas le commit correspondant et la recette jetable ne semble pas à jour (connecté à la belle verte).

EwenKorr · 2025-05-15T12:51:37Z

J'ai fait un commit fixup : fixup! add "edit" link for unauthorized orgs in side menu

En recette effectivement, ce n'est pas à jour (à cause des conflits ? Je vais résoudre ça)

francoisfreitag

Passé trop vite, j’ai vu le fixup et je me suis dit que c’était le même que lors de la dernière relecture 🙈

Le fixup me semble très bien.

EwenKorr · 2025-05-15T13:22:51Z

Merci ! La recette a été redéployée, ce devait être à cause du conflit.

The form is disabled, the buttons are useless.

As it is used in multiple apps and not only for companies.

…rganizations This content is never displayed.

hellodeloo added the ajouté Ajouté dans le changelog. label Apr 28, 2025

hellodeloo self-assigned this Apr 28, 2025

hellodeloo changed the title ~~feat: add overview page for prescribers~~ feat: ajout d'une page "presentation" pour l'organisation des prescripteurs habilités et orienteurs Apr 28, 2025

hellodeloo changed the title ~~feat: ajout d'une page "presentation" pour l'organisation des prescripteurs habilités et orienteurs~~ Prescripteur: ajout d'une page "presentation" pour l'organisation des prescripteurs habilités et orienteurs Apr 28, 2025

hellodeloo requested review from EwenKorr and francoisfreitag April 28, 2025 15:53

francoisfreitag reviewed Apr 29, 2025

View reviewed changes

Comment thread itou/templates/prescribers/members.html Outdated

Comment thread itou/templates/prescribers/overview.html Outdated

Comment thread itou/templates/prescribers/overview.html Outdated

Comment thread itou/www/prescribers_views/views.py

hellodeloo force-pushed the deloo/add-overview-prescribers-page branch 2 times, most recently from 6d4a13c to 8891b73 Compare April 30, 2025 09:45

hellodeloo added the 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC label Apr 30, 2025

hellodeloo force-pushed the deloo/add-overview-prescribers-page branch 2 times, most recently from 8a6deca to 9051456 Compare April 30, 2025 14:35

EwenKorr force-pushed the deloo/add-overview-prescribers-page branch 2 times, most recently from d051d74 to 641ab4a Compare May 2, 2025 13:10

EwenKorr force-pushed the deloo/add-overview-prescribers-page branch 6 times, most recently from 63fc1bd to 6b30840 Compare May 5, 2025 13:52

EwenKorr reviewed May 5, 2025

View reviewed changes

EwenKorr requested a review from francoisfreitag May 5, 2025 13:56

hellodeloo requested a review from EwenKorr May 5, 2025 13:59

francoisfreitag reviewed May 5, 2025

View reviewed changes

EwenKorr force-pushed the deloo/add-overview-prescribers-page branch 2 times, most recently from fbc77b6 to 997c499 Compare May 9, 2025 10:22

hellodeloo assigned EwenKorr May 12, 2025

francoisfreitag changed the title ~~Prescripteur: ajout d'une page "presentation" pour l'organisation des prescripteurs habilités et orienteurs~~ Prescripteur: ajout d'une page "presentation" pour l'organisation des prescripteurs habilités May 12, 2025

francoisfreitag reviewed May 12, 2025

View reviewed changes

EwenKorr force-pushed the deloo/add-overview-prescribers-page branch 2 times, most recently from cbf290f to 350e203 Compare May 12, 2025 13:55

francoisfreitag approved these changes May 12, 2025

View reviewed changes

EwenKorr force-pushed the deloo/add-overview-prescribers-page branch 4 times, most recently from ec32e12 to a0641e0 Compare May 15, 2025 12:05

EwenKorr force-pushed the deloo/add-overview-prescribers-page branch from a0641e0 to e700038 Compare May 15, 2025 12:54

francoisfreitag approved these changes May 15, 2025

View reviewed changes

EwenKorr added 5 commits May 16, 2025 08:34

prescribers: remove unused edit form buttons for FT organizations

e42f356

The form is disabled, the buttons are useless.

feat: add overview page for authorized prescribers

0f38c42

static: rename the illustration 'no-result'

4551d5c

As it is used in multiple apps and not only for companies.

prescribers: use back_url to redirect after editing the organization

a20160b

prescribers: remove Description field in edit form for unauthorized o…

c7fc535

…rganizations This content is never displayed.

EwenKorr force-pushed the deloo/add-overview-prescribers-page branch from e700038 to c7fc535 Compare May 16, 2025 06:34

EwenKorr changed the title ~~Prescripteur: ajout d'une page "presentation" pour l'organisation des prescripteurs habilités~~ Prescripteur : ajout d'une page "Présentation" pour l'organisation des prescripteurs habilités et d'une entrée dans le menu latéral May 16, 2025

EwenKorr added this pull request to the merge queue May 16, 2025

Merged via the queue into master with commit d6956ac May 16, 2025

EwenKorr deleted the deloo/add-overview-prescribers-page branch May 16, 2025 07:22

	# PE users often mistakenly edit this page, to the point where the
	# support asked to disable it, and have them reach out in case of
	# changes.

Uh oh!

Conversation

hellodeloo commented Apr 28, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

🤔 Pourquoi ?

Uh oh!

notion-workspace Bot commented Apr 28, 2025

Uh oh!

francoisfreitag left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

EwenKorr commented Apr 29, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

francoisfreitag commented Apr 29, 2025

Uh oh!

github-actions Bot commented Apr 30, 2025

Uh oh!

EwenKorr commented May 2, 2025

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

francoisfreitag left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

EwenKorr commented May 5, 2025

Uh oh!

francoisfreitag left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

EwenKorr May 12, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

EwenKorr commented May 12, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

EwenKorr commented May 15, 2025

Uh oh!

francoisfreitag commented May 15, 2025

Uh oh!

EwenKorr commented May 15, 2025

Uh oh!

francoisfreitag left a comment

Choose a reason for hiding this comment

Uh oh!

hellodeloo commented Apr 28, 2025 •

edited

Loading

EwenKorr commented Apr 29, 2025 •

edited

Loading

EwenKorr May 12, 2025 •

edited

Loading

EwenKorr commented May 12, 2025 •

edited

Loading