Skip to content

UI/UX: Remplacer PE par FT pour les agences France Travail [GEN-2058]#5545

Merged
tonial merged 3 commits into
masterfrom
alaurent/rename_pe_to_ft
Feb 7, 2025
Merged

UI/UX: Remplacer PE par FT pour les agences France Travail [GEN-2058]#5545
tonial merged 3 commits into
masterfrom
alaurent/rename_pe_to_ft

Conversation

@tonial

@tonial tonial commented Feb 4, 2025 •
edited
Loading

Copy link
Copy Markdown
Contributor

🤔 Pourquoi ?

Pour que les utilisateurs voient FT et non PE au dessus du nom de leur organisation.

⚠️ Déploiement à faire hors heures ouvrées pour minimiser les impacts !

Pas d'impact pour les données envoyées au pilotage
Pas de modification des données envoyées aux autres partenaires pour l'instant

🍰 Comment ?

Décrivez en quelques mots la solution retenue et mise en oeuvre, les difficultés ou problèmes rencontrés. Attirez l'attention sur les décisions d'architecture ou de conception importantes.

🚨 À vérifier

  • Mettre à jour le CHANGELOG_breaking_changes.md ?
  • Ajouter l'étiquette « Bug » ?

🏝️ 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

@notion-workspace

notion-workspace Bot commented Feb 4, 2025

Copy link
Copy Markdown

Remplacer PE par FT pour les agences France Travail

@tonial tonial self-assigned this Feb 4, 2025
@tonial tonial added the modifié Modifié dans le changelog. label Feb 4, 2025
@tonial tonial force-pushed the alaurent/rename_pe_to_ft branch 2 times, most recently from 99e1700 to b300ad6 Compare February 5, 2025 05:42
leo-naeka
leo-naeka reviewed Feb 5, 2025
View reviewed changes

@leo-naeka leo-naeka left a comment

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.

Effectivement faudra tirer au clair les FIXME.

Je doute que l'API France Travail ait mis à jour son endpoint pour supporter les deux valeurs, j'imagine qu'on va rester sur du PE un petit moment...

Idem pour metabase, t'as fait le choix de MAJ vers FT, critique pour eux ?

Comment thread itou/approvals/migrations/0010_alter_approval_origin_prescriber_organization_kind_and_more.py Outdated
Comment thread itou/fixtures/django/14_prescribers.json
Comment thread itou/prescribers/enums.py Outdated
@tonial

tonial commented Feb 6, 2025 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

Pour métabase, j'ai vu avec eux : pas d'impact car on leur envoie aussi l'enum pour faire la conversion donc ce sera transparent.

Pour FT, en effet, je ne vais pas changer un truc qui fonctionne.

J'ai donc retiré les FIXMEs

@tonial tonial force-pushed the alaurent/rename_pe_to_ft branch from b300ad6 to 9ca0607 Compare February 6, 2025 10:32
@tonial tonial requested a review from leo-naeka February 6, 2025 10:33
xavfernandez
xavfernandez reviewed Feb 6, 2025
View reviewed changes

@xavfernandez xavfernandez left a comment

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.

J'aurais bien vu 2 commits (au moins):

  • un pour le renommage pur-python de PrescriberOrganizationKind.PE -> PrescriberOrganizationKind.FT (et PrescriberType.PE -> PrescriberType.FT)
  • un autre pour la modification en base de données

@tonial tonial force-pushed the alaurent/rename_pe_to_ft branch from 9ca0607 to 299fa36 Compare February 6, 2025 12:43
@tonial

tonial commented Feb 6, 2025

Copy link
Copy Markdown
Contributor Author

J'en ai ajouté un en plus pour les renommages de fonctions/variables

@tonial tonial requested a review from xavfernandez February 6, 2025 13:08
xavfernandez
xavfernandez reviewed Feb 6, 2025
View reviewed changes
Comment thread tests/www/stats/test_utils.py Outdated
Comment on lines +126 to +133
def test_can_view_stats_ft_as_regular_fr_agency():
regular_fr_agency = PrescriberOrganizationWithMembershipFactory(
authorized=True, kind=PrescriberOrganizationKind.FT, department="93"
)
user = regular_pe_agency.members.get()
assert not regular_pe_agency.is_dtft
assert not regular_pe_agency.is_drft
assert not regular_pe_agency.is_dgft
user = regular_fr_agency.members.get()
assert not regular_fr_agency.is_dtft
assert not regular_fr_agency.is_drft
assert not regular_fr_agency.is_dgft

@xavfernandez xavfernandez Feb 6, 2025

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.

fr - > ft j'imagine ?

Comment thread tests/www/dashboard/test_dashboard.py
xavfernandez
xavfernandez approved these changes Feb 6, 2025
View reviewed changes

@xavfernandez xavfernandez left a comment

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.

Ça me semble bien 👍
Il faudra idéalement le déployer à un moment calme et loin d'un export vers metabase.

Il y a l'API GEIQ qui sera a priori également impacté: j'imagine qu'on renverra dorénavant FT au lieu de PE, je vais donner l'info à Zohra pour voir s'il faut mettre une rustine.

Est-ce que tu as également vu avec data-inclusion (cf

les-emplois/itou/api/data_inclusion_api/serializers.py

Line 130 in 86328e8

typologie = serializers.ChoiceField(source="kind", choices=PrescriberOrganizationKind.choices)
) ?

Comment thread itou/fixtures/django/21_approvals.json Outdated
"number": "XXXXX0000001",
"origin": "pe_approval",
"origin_prescriber_organization_kind": "",
"origin_prescriber_organization_kind": "FT",

@xavfernandez xavfernandez Feb 6, 2025

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.

C'est un peu étrange ça ?

@tonial tonial force-pushed the alaurent/rename_pe_to_ft branch from 299fa36 to 767a267 Compare February 6, 2025 15:10
@tonial

tonial commented Feb 6, 2025

Copy link
Copy Markdown
Contributor Author

Pour l'api Geiq, c'est inchangé : https://github.com/gip-inclusion/les-emplois/pull/5545/files#diff-1494e0292624f82b6faa651b5bb9fe65b02afe2879c1a9e3f07f991679a81f99L47-R47

Je demande à victor pour data inclusion

leo-naeka
leo-naeka approved these changes Feb 6, 2025
View reviewed changes

@leo-naeka leo-naeka left a comment

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.

Ça me semble bon.

Effectivement à déployer à un moment calme, mais tu l'avais déjà prévu 😌

@tonial tonial added this pull request to the merge queue Feb 7, 2025
Merged via the queue into master with commit 887f37b Feb 7, 2025
@tonial tonial deleted the alaurent/rename_pe_to_ft branch February 7, 2025 05:30
@rsebille rsebille mentioned this pull request Feb 10, 2025
Merged
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@leo-naeka leo-naeka leo-naeka approved these changes
@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.

3 participants

@tonial @leo-naeka @xavfernandez