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

La propriété "voie" est mal typée lors de l'édition d'un numéro dans toponyme #405

Closed
GllmR opened this issue Sep 16, 2021 · 2 comments · Fixed by #409
Closed

La propriété "voie" est mal typée lors de l'édition d'un numéro dans toponyme #405

GllmR opened this issue Sep 16, 2021 · 2 comments · Fixed by #409
Assignees

Comments

@GllmR
Copy link
Contributor

GllmR commented Sep 16, 2021

Sur la page principale, dans l'onglet "toponyme", lorsque l'on édite un numéro, une erreur console apparaît :
Warning: Failed prop type: Invalid prop initialValue.voie of type object supplied to NumeroEditor, expected string

Le problème vient du composant NumeroEditor, dans les propTypes, voie est typée comme "string" alors que c'est un objet. => Uniquement lorsqu'on édite un numéro depuis l'onglet toponyme.

Le composant attend voie._id qui est bien une string.

Une modification des propTypes devrait suffire à corriger le problème.

Le type de la propriété initialValue.voie n'est pas la même selon que le composant numeroEditor soit utilisé dans "voie" (string) ou dans "toponyme" (object)

@GllmR GllmR self-assigned this Sep 16, 2021
@GllmR
Copy link
Contributor Author

GllmR commented Sep 17, 2021

L'API-BAL ne renvoie pas le même format selon qu'on appelle getNumeros() ou getNumerosToponyme().

getNumerosToponyme() a besoin du nom de la voie pour l'afficher et classer les numéros par voie dans la partie front.

Solutions possibles :

  • Modifier l'API pour qu'elle renvoie un objet "voie" et adapter la partie front
  • Modifier l'API pour qu'elle renvoie une string avec l'id de la voie et faire un appel depuis le front pour récupérer les noms de voie à afficher.
  • Créer un autre composant NumeroEditor qui corresponde à l'objet renvoyé par getNumerosToponyme()

@tmerlier
Copy link
Contributor

On peut aussi faire en sorte que les propTypes acceptent les deux types si c'est justifié.

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 a pull request may close this issue.

2 participants