search

ApplETS / ETSMobile-Android2

Portail étudiant mobile regroupant les principaux services accessibles aux étudiants de l'École de technologie supérieure.
http://clubapplets.ca
Apache License 2.0
12 stars 11 forks source link

Sync bottin #92

Closed Sonphil closed 7 years ago

zaclimon commented 7 years ago

J'ai essayé de mon côté avec 15 minutes à la place de 1 jour et tout fonctionne comme il faut. Nice job! 👍 J'ai laissé un commentaire sur la job en soi mais il est beau pour être "merged" si nécessaire.

zaclimon commented 7 years ago

Le cas que j'avais soulevé assumait que la synchronisation ait été réussie dans le deuxième ou troisième tour.

Encore là, c'est plus une question de style de ma part. (Mon but n'étant pas de forcer une manière de faire mais plutôt de savoir pourquoi telle solution a été adopté hehe XD)

Je peux faire le merge mais pourrais-tu vérifier le #90 avant de faire cela?

Sonphil commented 7 years ago

Non, c'est bon. Ton approche fonctionne bien. J'avais mis les nombre de vérifications afin d'éviter que onRunJob s'exécute indéfiniment si DataManager s'éterniserait, mais ça ne semble pas arriver.

zaclimon commented 7 years ago

DataManager devrait lancer une exception s'il y a un problème lors de la synchronisation. (Timeout par exemple) Dans de telles circonstances, onRequestFailed() serait toujours appelé s'il y aurait quelque chose qui clocherait.