ApplETS / ApplETS/ETSMobile-iOS

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

Open
#123 0 comments 0 reactions 0 assignees View on GitHub
Corrective
Dominant language
Objective-C
Stars
8
Forks
3
PR merge metrics
No merged PRs in 30d

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."

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.