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

Corrige le message d'erreur sur le typage de token #406

Merged
merged 3 commits into from
Sep 22, 2021
Merged

Conversation

GllmR
Copy link
Contributor

@GllmR GllmR commented Sep 16, 2021

Cette PR corrige un message d'erreur dans la console en mode consultation :
Warning: Failed prop type: Invalid prop token of type boolean supplied to Publication, expected string.

À l'ouverture d'une Base Adresse Locale en mode consultation (sans jeton d'administration), le contexte du token vérifie que le jeton correspondant à la Base Locale est bien présent. S’il ne l'est pas, le token passait à false et était donc considéré de type "booléen".

Désormais, le jeton est défini à null si l'utilisateur n'est pas administrateur de la Base Locale consultée.

Fix #401

@GllmR GllmR requested a review from tmerlier September 16, 2021 12:57
@GllmR GllmR self-assigned this Sep 16, 2021
@vercel
Copy link

vercel bot commented Sep 16, 2021

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/livingdata/mes-adresses/Fgqhw1DZmjJR7MzRnksDoaimQ31Y
✅ Preview: https://mes-adresses-git-gllmr-token-bool-livingdata1.vercel.app

@tmerlier
Copy link
Contributor

Aucun endroit dans le code justifié le fait que token puisse être à false ? Cette valeur me pareil étrange et je crains qu'elle ne soit pas une erreur mais quelle serve un but spécifique.

@GllmR
Copy link
Contributor Author

GllmR commented Sep 21, 2021

Aucun endroit dans le code ne justifie le fait que token puisse être à false ? Cette valeur me paraît étrange et je crains qu'elle ne soit pas une erreur, mais qu'elle serve un but spécifique.

Il n'y a qu'un seul endroit où une comparaison est faite avec false , c'est ici.

Cette partie de code semble inutile puisqu'un utilisateur sans jeton ne peux pas afficher la partie paramètre, l'affichage du bouton étant conditionnée avec la présence ou non du jeton.

@tmerlier
Copy link
Contributor

Ok alors il faut s'assurer que ce code ne peut pas être exécuter et le supprimer si c'est bien le cas.

@tmerlier tmerlier merged commit a0b4089 into master Sep 22, 2021
@jdesboeufs jdesboeufs deleted the gllmr/token-bool branch January 17, 2022 12:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

En mode consultation, le token est de type booléen
2 participants