-
Notifications
You must be signed in to change notification settings - Fork 0
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
Feat: route search numero by polygon #474
Conversation
@@ -463,6 +465,29 @@ export class BaseLocaleController { | |||
res.status(HttpStatus.OK).json(allDeleted); | |||
} | |||
|
|||
@Put(':baseLocaleId/numeros') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Un put pour de la recherche ça me parait pas terrible, on peut pas passer le tableau de coordonnées dans l'URL du GET?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Aussi je suis surpris qu'on doive faire un endpoint pour ça. Y'a pas moyen de récupérer les numéros à l'intérieur d'un polygone directement côté front avec maplibre?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Alors j'ai utilisé un PUT parce que j'imagine que si la personne fait un polygon complexe (avec beaucoup de coordonées longues) la limit de 2000 caractère dans l'url peut sauter. Après c'est peut être pas utile a toi de me dire
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
On a pas de route qui retourne les numéros d'une bases_locales et on n peut pas se fier au tuile car il y a seulement les features (numeros) visiblent sur la carte qui sont chargés sur la carte.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hmm je comprends pas trop. De toute façon l'utilisateur va dessiner le polygone en fonction de ce qui est visible sur la carte.
Tu penses qu'il faut aussi ajouter les numéros supprimés ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Après ok pour le PUT même si ça me parait un peu overkill, je pense pas qu'on est des polygones qui dépassent les 20 points
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Il ne va pas forcement dessiner sur un endroit visible au moment du calcule, si il part sur un coté ou dézoom pour rajouter le dernier point. C'est un peut tirer par les cheveux mais ca peut arriver
FONCTIONNALITE
/bases-locales/:id
avec un polygon dans le body pour rechercher les numeros qui sont dans le polygon