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

Feat: route search numero by polygon #474

Merged
merged 114 commits into from
Oct 29, 2024

Conversation

fufeck
Copy link
Collaborator

@fufeck fufeck commented Sep 9, 2024

FONCTIONNALITE

  • Ajout d'une route PUT /bases-locales/:id avec un polygon dans le body pour rechercher les numeros qui sont dans le polygon

@fufeck fufeck changed the title Fufeck feat route search numero by polygon Feat: route search numero by polygon Sep 9, 2024
@fufeck fufeck requested a review from MaGOs92 October 21, 2024 14:02
@@ -463,6 +465,29 @@ export class BaseLocaleController {
res.status(HttpStatus.OK).json(allDeleted);
}

@Put(':baseLocaleId/numeros')
Copy link
Collaborator

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?

Copy link
Collaborator

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?

Copy link
Collaborator Author

@fufeck fufeck Oct 23, 2024

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

Copy link
Collaborator Author

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.

Copy link
Collaborator

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 ?

Copy link
Collaborator

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

Copy link
Collaborator Author

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

@fufeck fufeck requested a review from MaGOs92 October 23, 2024 17:02
@fufeck fufeck merged commit 73c8d76 into master Oct 29, 2024
3 checks passed
@fufeck fufeck deleted the fufeck_feat_route-search-numero-by-polygon branch October 29, 2024 09:56
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.

2 participants