lschaeffer313 / domaine-nc-mobile

Appli mobile pour la consultation des domaines en Nouvelle-Calédonie
https://play.google.com/store/apps/details?id=nc.laurent.domaine_nc_mobile&pli=1
GNU Affero General Public License v3.0
7 stars 2 forks source link

🚨 Crash de l'application sur les domaines protégé 🚨 #21

Closed lschaeffer313 closed 1 year ago

lschaeffer313 commented 1 year ago

🐛 Problème

Lorsqu'on consulte un domaine qui est protégé (du style gouvernement.nc), l'application ne répond plus.

👉 Cause

L'application s'attend à ce que l'API de domaine-nc-api retourne un objet avec des champs non null par l'endpoint /name/extension.

😵 Solution temporaire

Rendre les champs nullables afin que l'application ne rentre plus en erreur.

👍 Solution pérenne dans le temps

Faire en sorte que l'API renvoi des champs non-nullable. On pourrait imaginer qu'il renvoi des chaînes de caractères vides.

✊ Action à faire dans tout les cas

Prendre en compte le cas des domaines protégé dans l'application.

❓Pourquoi cette issue ?

Dart implémente ce qu'on appelle le null safety ✅. Cela prémunit d'accéder à des références d'objet ou de champs qui sont null. Cela incite (ou plutôt oblige) le développeur a écrire du code sans risque d'avoir des erreurs de type NullPointerException.

lschaeffer313 commented 1 year ago

Les champs en question : image

lschaeffer313 commented 1 year ago

@adriens je créer une issue sur domaine-nc-api ?

adriens commented 1 year ago

@adriens je créer une issue sur domaine-nc-api ?

:rofl: J'allais te demander, yes :pray:

lschaeffer313 commented 1 year ago

@adriens ticket mise à jour suite à notre discussion 😁

adriens commented 1 year ago

@adriens ticket mise à jour suite à notre discussion grin

Excellent :muscle:

spencer-forrest-opt commented 1 year ago

Dart implémente ce qu'on appelle le null safety ✅

@adriens, @3079rod un peu comme KOTLIN, non ?  😆

lschaeffer313 commented 1 year ago

@spencer-forrest-opt Exactement

adriens commented 1 year ago

Dart implémente ce qu'on appelle le null safety ✅

@adriens, @3079rod un peu comme KOTLIN, non ? 😆

LOOOOOOOL ENORME !!!! OUIIIIII, excellent @spencer-forrest-opt @3079rod @sverolle 🤣 🤣 🤣 🤣 🤣 🤣

adriens commented 1 year ago

:clap: