search

entrepreneur-interet-general / CIS-front

Site public du carrefour des innovations sociales
http://carrefourdesinnovationssociales.fr/
MIT License
7 stars 5 forks source link

Intégration des illustrations des pages d'erreur #191

Closed Eliselalique closed 5 years ago

Eliselalique commented 5 years ago

Lié à l'issue #122

Trois écrans d'erreur à intégrer : (illustrations dispo sur github)

Pas de résultats (moteur de recherche) moteur de recherche_pas_de_resultats

Erreur 404 erreur_404

Erreur 500 erreur_500

JulienParis commented 5 years ago

remplace l'issue #82

DavidBruant commented 5 years ago

Je réalise qu'il manque une maquette pour les erreurs 403 (quand une personne n'a pas le droit d'avoir accès à une URL, typiquement parce qu'il faut un compte pour la voir) Dans l'attente, je vais mettre la même page que 404 (ce qui permet de réduire des fuites de données d'ailleurs)

DavidBruant commented 5 years ago

@Eliselalique est-ce que les pages d'erreur ont un footer ?

DavidBruant commented 5 years ago
Eliselalique commented 5 years ago

Ok pour la 403 je vais la faire :)

Oui pour le footer

Le mar. 22 janv. 2019 à 15:08, David Bruant notifications@github.com a écrit :

  • Gérer les erreurs côté serveur
  • Gérer les erreurs 404 aussi depuis le routeur côté client

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/entrepreneur-interet-general/CIS-front/issues/191#issuecomment-456411862, or mute the thread https://github.com/notifications/unsubscribe-auth/AilOMsXba0d5QSqMglx0h9DX0yidvfebks5vFxt2gaJpZM4ZLOkL .

-- Elise Lalique

DavidBruant commented 5 years ago

@Eliselalique J'ai ça actuellement Est-ce que je laisse la barre avec le zéro et les liens vers carte/données ou je les vire comme le suggère la maquette plus haut ?

capture d ecran de 2019-01-22 16-09-46

DavidBruant commented 5 years ago

(je vais me débrouiller pour que le bouton n'apparaisse que de manière conditionnelle seulement quand des filtres sont choisis)

Eliselalique commented 5 years ago

Si c'est galère à enlever laisse-les

Le mar. 22 janv. 2019 à 16:11, David Bruant notifications@github.com a écrit :

(je vais me débrouiller pour que le bouton n'apparaisse que de manière conditionnelle seulement quand des filtres sont choisis)

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/entrepreneur-interet-general/CIS-front/issues/191#issuecomment-456434671, or mute the thread https://github.com/notifications/unsubscribe-auth/AilOMvFp3Ikv2ok8ahpOJ8tkLsix1ed4ks5vFyo4gaJpZM4ZLOkL .

-- Elise Lalique

DavidBruant commented 5 years ago

Si c'est galère à enlever laisse-les

Suite à discussion, je laisse comme ça, tu testes et tu fais une nouvelle issue si tu veux que ça soit retiré (c'est facile)

Aussi, les 403, ça se passe ici : #267