Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Closed
5 tasks done
odelcroi opened this issue Jan 26, 2024 · 8 comments
Closed
5 tasks done

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

odelcroi opened this issue Jan 26, 2024 · 8 comments

Comments

@odelcroi
Copy link
Member

odelcroi commented Jan 26, 2024

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

  • 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 :
Capture d’écran 2024-01-26 à 11 01 27

On peut déclencher cette dialogue lors de la fin de l'appel :
Capture d’écran 2024-01-26 à 11 02 02

How

Tasks

Preview Give feedback
  1. P1 web
  2. yostyle

Alternative

  • créer un dialog inspiré de BugReportDialog avec des champs à rémplir pour etre sur que le user nous enverra des data formatté
@odelcroi odelcroi changed the title Voip : signaler une anomalie lors d'un appel avec les logs Voip : signaler une anomalie (avec logs) lors d'un appel avec les logs Jan 26, 2024
@odelcroi odelcroi changed the title Voip : signaler une anomalie (avec logs) lors d'un appel avec les logs Voip : signaler une anomalie (avec logs) lors d'un appel Jan 26, 2024
@NicolasBuquet
Copy link

@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
Copy link

yostyle commented Jan 29, 2024

share_3007283084374592937

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

@odelcroi odelcroi moved this to sprint en cours - WIP in Roadmap Produit Jan 30, 2024
@areox-net
Copy link

areox-net commented Jan 31, 2024

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;
Image

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
Image

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é.
Image

@odelcroi
Copy link
Member Author

odelcroi commented Feb 5, 2024

ajouter dans le rageshake :

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

@NicolasBuquet
Copy link

NicolasBuquet commented Feb 5, 2024

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
Copy link

yostyle commented Feb 5, 2024

On pourrait en dire autant sur le type de connexion.

@odelcroi odelcroi added the epic label Feb 6, 2024
@odelcroi odelcroi moved this from sprint en cours - WIP to beta phase / infra à déployer in Roadmap Produit Feb 14, 2024
@estellecomment
Copy link

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
Copy link

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. tchapgouv/tchap-web-v4#917

@github-project-automation github-project-automation bot moved this from beta externe to déployé en prod in Roadmap Produit Mar 19, 2024
@odelcroi odelcroi moved this from déployé en prod to done in Roadmap Produit May 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: done
Development

No branches or pull requests

5 participants