CAUCA-9-1-1 / survip-mobile

SURVI-Prevention Mobile
4 stars 1 forks source link

Probleme avec la sauvegarde de photo #278

Closed jocelyncardinal closed 5 years ago

jocelyncardinal commented 5 years ago

Exemple ...

Fonctionne pas pour l'image suivante ...

Capture d’écran 2018-11-27 à 14.49.52.png

par contre celle-ci fonctionne ...

Capture d’écran 2018-11-27 à 14.51.32.png

jocelyncardinal commented 5 years ago

Audrey, je vais t'envoyer les deux images par courriel pour test

jocelyncardinal commented 5 years ago

Je note aussi problème avec sauvegarde des annotations ...

Capture d’écran 2018-11-27 à 15.03.51.png

iamkinetic commented 5 years ago

Ça fait ça à tout coup avec ces deux images?

iamkinetic commented 5 years ago

J'ai vu la première dans le plan d'implantation de mon inspection de test hier maintenant que j'y pense.

jocelyncardinal commented 5 years ago

Oui, je viens de retester et présentement ca bug a tout coup sur mon Iphone pour la premiere image et ca fonctionne a tout coup pour la 2e. Toutefois, il est possible que j'aille utilisé cette image (la 1ere) par le passé et que ca avait fonctionné a ce moment la et c'est pour ca que tu en l'a vue dans des inspection. Note que ca fonctionne pour TOUTES les images sur le télephone Android a Audrey (elle n'est pas capable de reproduire le problème. Ca l'air d'etre relié au Iphone Pour l'inspection que tu as vue avec l'image No1, elle vient probablement de tests que Audrey a fait (ou que jai fait précédemment et que l'image a réussi a etre sauvegarder a ce moment la). Je vais te transférer les deux images pour voir si vous pouvez reproduire le probleme sur vos Iphones de test

AudreyLR commented 5 years ago

Je confirme que ça fonctionne de mon côté sur Android, ainsi que sur simulateur iphone dans mon environnement de développement. Par contre, je n'ai pas de iPhone pour tester, alors je n'ai pas pu reproduire ce bug.

iamkinetic commented 5 years ago

@philipperobertgh Peux-tu voir si tu peux reproduire le problème avec un des iPhones du bureau?

iamkinetic commented 5 years ago

@JocelynCardinal Nous sommes en train de modifier la configuration du serveur de test, donc il se peut que tu aies des erreurs dans les prochaines minutes.

iamkinetic commented 5 years ago

Asp.net core a une limite par défaut de 30mb par requête. NGInx a quant à lui une limite de 1mb par requête. Notre théorie est que NGinx est configuré avec la limite par défaut ce qui fait que certaines photos ne passent pas parce qu'elles dépassent le 1mb. Nous attendons que le tech qui s'occupe de ça soit libre pour confirmer le tout.

iamkinetic commented 5 years ago

@philipperobertgh Il faudra réfléchir à la façon de mieux gérer ça pour empêcher l'utilisateur d'essayer de mettre des photos trop grosses et de gérer correctement l'erreur quand on essaie d'ajouter une photo et que ça ne passe pas au niveau de l'API.

Et il faudrait enlever la configuration qui force un redimensionnement sur le mobile (ionic-camera).

philipperobertgh commented 5 years ago

Le redimensionnement de l'image à la selection sur le disque n'est pas soumis aux mêmes resolutions selon le téléphone, c'est pour cela que cette situation ne se passe actuellement que sur IPhone. Nous allons ajuster la limite de transfert concernant les images. et mettre une validation pour les usagers si les images qu'ils tentent d'envoyer sont considérées comme trop volumineuses.

philipperobertgh commented 5 years ago

Le service technique est actuellement en vérification du problème de blocage de transfert de fichier vers le serveur de test.

philipperobertgh commented 5 years ago

Le service technique a augmenter la limite réseau pour les fichier volumineux, tester avec la photo transmise par Jocelyn. Tout est fonctionnel. @iamkinetic Doit-on faire les modifications de validation de suite ou on créér un autre point car cela ne bloque en rien le fonctionnement de l'application ?

iamkinetic commented 5 years ago

On peut faire un point avec le mobile pour bloquer les trop gros fichiers et augmenter la résolution de l'image (pour passer de 600x480 à une plus grosse résolution). On pourrait faire des tests pour trouver une configuration qui a du sens : on peut jouer avec la résolution et le % de qualité et le format (jpg, png). On devrait vérifier si on pourrait enlever la limite de résolution, mais baisser la qualité à 80% pour trouver un juste milieu.

Actuellement on est à 800x680, JPEG, 100%.

Et on devrait aussi créer un point pour le site web pour s'assurer que les images qu'on y ajoute ait les même limitations que sur mobile.

jocelyncardinal commented 5 years ago

Je confirme que ça fonctionne bien maintenant sur mon iPhone 8 Testé plusieurs grosses images dans plan d’intervention et anomalies avec succès .

Good job!!!