Com 2115

[KennyCallegari]

• J'ai testé sur iphone

• J'ai testé sur android

• La nouvelle version de l'app est compatible avec l'ancienne version de l'API ? (J'ai teste en me mettant sur
  master en api)

  • Oui parce que la MEP intermédiaire a été faite
  • Non parce que

• Je n'envoie pas de nouveau paramètre dans une route

  • Si j'en envoie un nouveau, explication et gestion de la compatibilite:

• J'attends un nouveau champs en retour de l'api: j'ai géré le cas où il n'y est pas

• J'appelle une nouvelle route: j'ai géré le cas ou la route n'existe pas.

• Je n'ai pas changé de constante

• J'ai ajouté une variable d'environnement :

  • Je l'ai ajouté dans env.dev et env.prod aussi

Cas d'usage :
Fix de 3 bugs remontés durant les tests des notifs de la semaine dernière :

• sur Android : Lorsque je ferme complètement l'app, que je reçois une notification et que je clique dessus, je suis bien emmené vers la page A Propos de la formation correspondante.
  C'est un bug connu de expo mais qu'ils n'ont pas l'air d'avoir corrigé : [unimodules][android][notifications] Event listener can miss emit() calls made soon after app launch expo/expo#9866, [notifications] selecting notification when app is killed should trigger NotificationResponseReceivedListener expo/expo#6943
  J'ai utilisé un fix donné par un des gestionnaire d'Expo mais j'ai peur de mal gérer mes listeners du coup : https://gist.github.com/cruzach/eadbf47990e57b9ff4c808dd377d44c7
  J'ai aussi trouvé une autre solution mais elle ne fonctionne pas dans ce cas non plus : https://docs.expo.io/versions/latest/sdk/notifications/#handling-push-notifications-with-react-navigation
  UPDATE :
  J'ai changé la méthode pour gérer les notifications, maintenant ça fonctionne et sans risque de fuite mémoire.
  https://docs.expo.io/versions/latest/sdk/notifications/#uselastnotificationresponse-undefined--notificationresponse--null

• sur Android, l'icône d'app sur la notification est un carré gris : c'est un problème connu je l'ai donc corrigé. En gros, il faut une icône d'app blanche sur fond transparent et d'une certaine taille sinon on a un carré gris.
  Malheureusement, on ne pourra tester ça que sur TestFlight, en attendant, on a l'icône d'Expo Go
  Push notification icon is a gray square expo/expo#11579

• On ne demandait pas l'autorisation / on n'enregistrait pas l'expoToken dans le cas où l'utilisateur était connecté + avait son app ouverte au moment de la mise-a-jour. Je l'ai corrigé et pour tester, il faut passer de la v1.10.0 à ma branche en local en ayant l'app ouverte + en étant connecté. Et on vérifie qu'en bdd, on a bien l'expoToken qui est ajouté.

1 bug n'a pas été fix. Il se décompose en 3 cas :

• si l'utilisateur bloque les notifications et qu'il ne revient pas sur l'app, son expoToken n'est pas supprimé en bdd
• si l'utilisateur débloque les notifications et qu'il ne revient pas sur l'app, son expoToken n'est pas enregistré en bdd
• si l'utilisateur supprime l'app de son téléphone, , son expoToken n'est pas supprimé en bdd

Pour le 3ème cas, ce n'est pas grave : l'expoToken n'est plus relié à un téléphone donc on a une erreur à l'envoi de la notification. On pourra supprimer les expoToken dans ce cas là lorsqu'on mettra en place l'amélioration de la gestion de la reception des notifications en back.
Vous pouvez tester ce cas avec l'outil expo : https://expo.io/notifications avec le token de l'app expo go que vous venez de supprimer

Pour les deux premiers cas, je ne vois pas comment faire.
Pour le 1er cas, j'avais lu que ce n'est pas conseillé par Apple mais je ne retrouve pas cette info et je n'ai pas vu de gros warning sur leur doc.
https://developer.apple.com/library/archive/documentation/NetworkingInternet/Conceptual/RemoteNotificationsPG/BinaryProviderAPI.html#//apple_ref/doc/uid/TP40008194-CH13-SW1
Par contre, il est impossible d'avoir un listener sur cette information, donc même si on avait l'app ouverte au moment où l'utilisateur coupe ou remet les notifications sur son téléphone, on ne pourrait rien y faire. Il faut attendre qu'il revienne sur l'app.
https://developer.apple.com/forums/thread/69101

[KennyCallegari]

⚠️ cette méthode créée-t-elle une possibilité de fuite mémoire ?

[ulysseferreira]

expo aurait proposer quelque chose dans sa doc pour fermer le listener si c'était le cas non ?