#patch (2644) [Cadastre] Nuancer les erreurs lors de la recherche de propriétaires fonciers#1445
Merged
superfeedboy merged 3 commits intodevelopfrom Mar 25, 2026
Merged
#patch (2644) [Cadastre] Nuancer les erreurs lors de la recherche de propriétaires fonciers#1445superfeedboy merged 3 commits intodevelopfrom
superfeedboy merged 3 commits intodevelopfrom
Conversation
… propriétaires fonciers refers to https://trello.com/c/5enKQwsi/2644 - Ajout de codes d'erreur spécifiques dans majicOwners/find.ts : * parcel_fetch_error : erreur SQL lors de la recherche de parcelle * parcel_fetch_failed : parcelle introuvable en base * owners_fetch_error : erreur SQL lors de la recherche de propriétaires * owners_fetch_failed : propriétaires introuvables en base
refers to https://trello.com/c/5enKQwsi/2644 Lors de la recherche d'un propriétaire foncier : - Erreurs "not found" (parcel_fetch_failed, owners_fetch_failed) : → notification de type INFO avec titre "Information introuvable" - Erreurs techniques (SQL, permissions, envoi mail, etc.) : → notification de type ERROR avec titre "Une erreur est survenue" La distinction se fait via le test du code d'erreur retourné par l'API plutôt que par analyse du message (plus robuste et maintenable).
refers to https://trello.com/c/5enKQwsi/2644 - Suppression du console.error dans shantytownParcelOwner/update.ts (déjà géré par ServiceError)
Contributor
✅ Snyk checks have passed. No issues have been found so far.
💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse. |
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
🧾 Ticket Trello
https://trello.com/c/5enKQwsi/2644
🛠 Description de la PR
Objectif
Améliorer l'expérience utilisateur en distinguant les erreurs techniques des cas où les données recherchées sont simplement introuvables dans la base des propriétaires fonciers.
Modifications
Backend
Nouveaux codes d'erreur dans majicOwners/find.ts:
parcel_fetch_error: erreur SQL/système lors de la recherche de parcelleparcel_fetch_failed: parcelle non trouvée en base (résultat vide)owners_fetch_error: erreur SQL/système lors de la recherche de propriétairesowners_fetch_failed: propriétaires non trouvés en base (résultat vide)Architecture :
try/catch: capture les erreurs SQL/systèmeif (!result): détecte les résultats videscoded'erreur dans la réponse HTTPFrontend
Notifications différenciées dans ModaleConnaitreProprietaire.vue:
info🔵error🔴📊 Cas d'usage
Avant
Tous les cas affichaient une notification d'erreur rouge "Une erreur est survenue"
Après
44123456AB0001inexistante → notification bleue "Information introuvable"🚨 Notes pour la mise en production