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

Correction cas éligibilité intermédiaires + commentaires selon spéc #896

Merged
merged 1 commit into from
Oct 16, 2024

Conversation

totakoko
Copy link
Member

Revoit les cas de gestion selon la spécification.

Pour passer de
image
à
image

Pour le futur futur, il faudrait que l'API retourne toutes les infos (réseau existant, en construction, PDP etc), pour que l'UI ait exactement la structure de la spéc.

@totakoko
Copy link
Member Author

@@ -322,6 +323,7 @@ export const getEligilityStatus = async (lat: number, lon: number, city?: string
hasNoTraceNetwork: null,
};
}
// Réseau futur à moins de 100m (60 sur Paris)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

60m pour etre cohérent avec au dessus

@@ -340,6 +342,7 @@ export const getEligilityStatus = async (lat: number, lon: number, city?: string
};
}

// Dans zone futur réseau
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pas tres francais ou alors "futur réseau" entre guillements

@@ -394,6 +437,8 @@ export const getEligilityStatus = async (lat: number, lon: number, city?: string
hasNoTraceNetwork: null,
};
}

// Pas de tracé sur la ville, mais ville où l’on sait qu’existe un réseau (repère)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pas de tracé sur la ville, mais ville dans laquelle on sait qu’il existe un réseau (repère)

Copy link
Member Author

@totakoko totakoko Oct 16, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

En fait pour tous les termes, j'ai copié collé tel quel deux du doc, cf ticket trello. Mais je suis d'accord avec toi que ça pourrait être plus lisible.
Futur réseau => réseau en construction par exemple

Théoriquement, je pousse Florence à faire une page Notion afin d'avoir les différents cas toujours à jour. Et ça sera plus facile à modifier également.

Tu préfères que je corrige les choses maintenant ou on attend la mise à jour côté Florence ?

Note : On peut faire ça dans une autre PR pour déployer ce fix en prod.

@totakoko totakoko merged commit 30fa918 into dev Oct 16, 2024
5 checks passed
@totakoko totakoko deleted the correction_eligibilite_cas_futur_proche_reseau_loin branch October 16, 2024 14:21
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