Voip : signaler une anomalie (avec logs) lors d'un appel

odelcroi commented 5 months ago

Je veux faire un appel Tchap 1-1, mais ya un truc qui se passe mal

Why

Donner un accès rapide aux user pour nous signaler un problème lors d'un appel Tchap, tout en nous fournissant les logs clients

[x] est ce que ya des trucs intéressants les logs, sinon, ben ignorez ce ticket :D

What

Envoyer un rageshake pré-rempli avec les logs, par exemple :

On peut déclencher cette dialogue lors de la fin de l'appel :

How

### Tasks
- [x] sur web, c'est possible de préremplir du texte dans la dialog grâce à la prop [initialText de la classe BugReportDialog](https://github.com/matrix-org/matrix-react-sdk/blob/38f791b79db57a7b977bf3c4ce69559ea223ee53/src/components/views/dialogs/BugReportDialog.tsx#L66)
- [ ] https://github.com/tchapgouv/tchap-web-v4/issues/888
- [ ] https://github.com/tchapgouv/tchap-ios/pull/964
- [ ] https://github.com/tchapgouv/tchap-android/pull/1010
- [x] https://github.com/tchapgouv/matrix-rageshake-crisp/pull/44/files

Alternative

créer un dialog inspiré de BugReportDialog avec des champs à rémplir pour etre sur que le user nous enverra des data formatté

NicolasBuquet commented 5 months ago

@odelcroi @yostyle Sur iOS, on peut exposer le champ d'input du RageShake et on pourra remplir son contenu avec du custom VoIP.

Niveau projet Xcode, on est dans /Riot/Module/BugReport, donc ça ne pose aucun problème.

yostyle commented 5 months ago

share_3007283084374592937

Sur android je peux créer un formulaire dédié à la VoIP en repartant du formulaire de bug report

areox-net commented 5 months ago

Je trouve le design Element des box de notif appel trop encombrant, surtout dans une logique 1-to-1, et pas assez explicite: appel manqué, non repondu et terminé, tout est pareil;

Utilisons le meme bouton CTA "Rappeler" pour déclancher le formulaire RageShake

Proposition

A terme je propose de changer l'aspect de ces box comme ça

Mais dans un premier temps, et dans le cadre du test DINUM, on peut se contenter d'enlever l'image profil et colorer le picto appel manqué.

odelcroi commented 4 months ago

ajouter dans le rageshake :

[ ] MUST : "context" : "voip"
[ ] NICE : "connection" : "wifi/4g/vpn"
[ ] NICE : "audio_input" : "headset_bluetooth" ?

NicolasBuquet commented 4 months ago

L'option RageShake "audio_input" n'est en fait pas bonne : quand on fait le RageShake, l'appel VoIP est terminé.

D'une part, on est post-appel.

D'autre part, on n'a plus d' AVAudioSession (iOS) active pour interroger les ports/connexions.

yostyle commented 4 months ago

On pourrait en dire autant sur le type de connexion.

estellecomment commented 4 months ago

Pour web : J'ai investigué comment savoir si ya un casque BT. On peut le savoir que sur Chrome, ya pas les API dispo sur firefox et edge. Donc je vais faire 3 valeurs du champ audio_input : "headset_bluetooth", "device" et "unknown". Parce que ya qd meme pas mal de unknown et ca va fausser les stats.

Et ya le meme pb que sur mobile : ca nous dira s'il y a un casque au moment où on envoie le rageshake, pas au moment de l'appel.

estellecomment commented 4 months ago

PR mergée pour web ! Finalement j'ai fait "headset_bluetooth" et "device", et quand on sait pas c'est "device". Sachant qu'on ne sait que sur Chrome, et encore pas à tous les coups, je sais pas pourquoi. J'ai fait une issue, et s'il s'avere que c'est important de vraiment savoir, on pourra s'y remettre. https://github.com/tchapgouv/tchap-web-v4/issues/917

tchapgouv / tchap-product