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

Feature : Enregistrement manuel dans la modale avec choix d'une ville et non d'une région #1284

Open
wants to merge 87 commits into
base: release/siret-cross-recording
Choose a base branch
from

Conversation

oumeimaelisbihani
Copy link
Collaborator

@oumeimaelisbihani oumeimaelisbihani commented Nov 6, 2024

Infos : j'ai mis en branche de base la branche modal pour éviter les diff inutiles

Changements réalisés :

  • Utilisation de l'api https://geo.api.gouv.fr/ avec la création d'une LocalisationApi
  • Modification du composant TeeProfileLocalisation
  • Ajout de types GeoResult et ConvertedGeoResult correspondant aux résultats de l'api geogouv
  • Ajout d'un utils localisation pour séparer les communes par code postal (le cas de paris par exemple)
  • Ajout de la fonction getManualCompanyData pour plus de lisibilité dans l'enregistrement
  • Ajout de la fonction convertLocalisation pour récupérer uniquement les infos similaires au cas enregistrement siret (region, ville, codePostal)

Tests réalisés :
Sur la recherche :

  • pas de résultat >> message d'erreur affiché
  • modification de l'input >> mise à jour des résultats
  • suppression de l'input >> affichage de tout les résultats

Sur l'enregistrement des données :

  • pas d'enregistrement de données possible si pas de ville sélectionnée
  • affichage des données enregistrées après un refresh

@oumeimaelisbihani oumeimaelisbihani self-assigned this Nov 6, 2024
@oumeimaelisbihani oumeimaelisbihani changed the base branch from main to release/siret-cross-recording November 6, 2024 16:27
@oumeimaelisbihani oumeimaelisbihani linked an issue Nov 6, 2024 that may be closed by this pull request
@oumeimaelisbihani oumeimaelisbihani marked this pull request as ready for review November 6, 2024 16:31
@oumeimaelisbihani oumeimaelisbihani changed the base branch from release/siret-cross-recording to feat/modal-compagny-data November 6, 2024 16:33
@oumeimaelisbihani oumeimaelisbihani marked this pull request as ready for review November 7, 2024 10:59
@ttdm ttdm removed the request for review from dolemoine November 7, 2024 12:56
@ttdm
Copy link
Collaborator

ttdm commented Nov 8, 2024

Suite à nos discussions d'hier sur l'utilisation d'un fichier texte vs api, je me suis supprimé de la review.
Je te laisse me retag/nous retag avec dorian quand ce sera ok.

@ttdm ttdm removed their request for review November 13, 2024 20:40
Base automatically changed from feat/modal-compagny-data to release/siret-cross-recording November 15, 2024 16:14
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.

Modification de la commune dans la modale entreprise
3 participants