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

Profil salarié: bascule vers une nouvelle URL & utilisation de l'identifiant publique du salarié dans l'URL #4713

Merged
merged 3 commits into from
Sep 11, 2024

Conversation

xavfernandez
Copy link
Contributor

🤔 Pourquoi ?

On aura bientôt une vrai page de détail de PASS IAE et la page salarié actuelle avec son URL en /approvals/detail/{PK de PASS} n'est pas cette page.

Elle est maintenant située en /employees/detail/{public_id de salarié}.

J'ai préféré laisser la vue liste en l'état pour ne pas tout déplacer d'un coup mais en cible il faudrait également adapter ApprovalListView.

🍰 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 ?

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

@xavfernandez xavfernandez added the no-changelog Ne doit pas figurer dans le journal des changements. label Sep 10, 2024
@xavfernandez xavfernandez self-assigned this Sep 10, 2024
Copy link
Contributor

@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.

Les tests couvrent tellement bien ❤️

@xavfernandez xavfernandez added the 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC label Sep 11, 2024
Copy link

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

@xavfernandez xavfernandez added this pull request to the merge queue Sep 11, 2024
Merged via the queue into master with commit 58dc7c8 Sep 11, 2024
15 checks passed
@xavfernandez xavfernandez deleted the xfernandez/employee_views branch September 11, 2024 09:10
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 no-changelog Ne doit pas figurer dans le journal des changements.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants