ApplETS / ApplETS/ETSMobile-iOS

Modifier ETSSynchronisation pour gérer les échecs d'accès au réseau

Ouverte
#123 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub
Corrective
Langage dominant
Objective-C
Étoiles
8
Forks
3
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

Actuellement, un échec d'accès au réseau ne charge aucune erreur et laisse l'utilisateur sans information.
Il faudrait détecter le NSError de l'appel réseau, et l'envoyer au Delegate pour le gérer dans la vue.

Voilà ce que Jean-Philippe Martin en dit :
"Tu me fais réaliser que j’ai mis un NSError dans la fonction synchronize mais qu’en aucun cas il va retourner l’erreur. Pour corriger le problème, il faudrait renommer la fonction du protocole synchronizationDidFinishLoading, pour y inclure le NSError en paramètre, et à la réponse de la requête HTTP, s’il y a erreur, appeler cette fonction delegate avec l’erreur retourné par le dataTaskWithRequest. Il y a d’ailleurs un // FIXME à cet endroit. Toutes les erreurs HTTP sont présentement ignorées."

Guide de contribution

Aucun guide de contribution indexé pour ce dépôt

Évaluation

Cette issue n'a pas encore été évaluée.

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.