Skip to content

GPS: correction de deux erreurs d'affichage (nom du candidat et nom de l'utilisateur connecté)#4263

Merged
celine-m-s merged 4 commits intomasterfrom
celinems/fix-bugs
Jun 21, 2024
Merged

GPS: correction de deux erreurs d'affichage (nom du candidat et nom de l'utilisateur connecté)#4263
celine-m-s merged 4 commits intomasterfrom
celinems/fix-bugs

Conversation

@celine-m-s
Copy link
Contributor

@celine-m-s celine-m-s commented Jun 19, 2024 •
edited
Loading

🤔 Pourquoi ?

  • Dans la page de détails d'un bénéficiaire, dans le menu déroulant de l'utilisateur connecté, le nom n'était plus celui de l'utilisateur connecté mais celui du bénéficiaire.
  • Les membres d'un groupe de suivi voyaient uniquement les initiales du bénéficiaire.

🍰 Comment ?

Distinction des variables.

@celine-m-s celine-m-s added the modifié Modifié dans le changelog. label Jun 19, 2024
@celine-m-s celine-m-s self-assigned this Jun 19, 2024
@celine-m-s celine-m-s mentioned this pull request Jun 19, 2024
Merged
3 tasks
@celine-m-s celine-m-s marked this pull request as ready for review June 19, 2024 15:08
@celine-m-s celine-m-s requested a review from tonial June 19, 2024 15:09
@notion-workspace
Copy link

notion-workspace bot commented Jun 19, 2024

Correction d’un bug

tonial
tonial reviewed Jun 20, 2024
View reviewed changes
Copy link
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.

Est-ce que ça ne vaut pas le coup de mettre un mini test ?

@celine-m-s
Copy link
Contributor Author

celine-m-s commented Jun 20, 2024

👍 J'en ai profité pour créer un test spécifique à cette vue.
Le commit sera bien sûr intégré dans le précédent.

@celine-m-s celine-m-s requested a review from tonial June 20, 2024 10:48
tonial
tonial reviewed Jun 20, 2024
View reviewed changes
Copy link
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.

J'ai l'impression que tu ne vérifies pas que ce sont les informations du bénéficiaire qui sont affichée,
Est-ce que tu ne peux pas utiliser directement snapshot, en utilisant les for_snapshot pour fixer les informations du prescripteur et du bénéficiaire ?

@celine-m-s celine-m-s requested a review from tonial June 21, 2024 08:22
@celine-m-s
Copy link
Contributor Author

celine-m-s commented Jun 21, 2024

J'ai l'impression que tu ne vérifies pas que ce sont les informations du bénéficiaire qui sont affichée,
Est-ce que tu ne peux pas utiliser directement snapshot, en utilisant les for_snapshot pour fixer les informations du prescripteur et du bénéficiaire ?

Tu as raison. J'ai simplement repris l'existant mais il n'y avait pas de test concernant les détails du bénéficiaire. C'est modifié !

@celine-m-s celine-m-s changed the title GPS: correction d'une erreur d'affichage GPS: correction de deux erreurs d'affichage (nom du candidat et nom de l'utilisateur connecté) Jun 21, 2024
@celine-m-s celine-m-s enabled auto-merge June 21, 2024 08:34
@celine-m-s celine-m-s added this pull request to the merge queue Jun 21, 2024
Merged via the queue into master with commit b71d9fc Jun 21, 2024
@celine-m-s celine-m-s deleted the celinems/fix-bugs branch June 21, 2024 08:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@tonial tonial tonial approved these changes

Assignees

@celine-m-s celine-m-s

Labels

modifié Modifié dans le changelog.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@celine-m-s @tonial