Projet

Général

Profil

Development #69972

phonecalls: sur un callstart d'un appel déjà existant, créer un nouvel appel

Ajouté par Thomas Noël il y a plus d'un an. Mis à jour il y a plus d'un an.

Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Version cible:
-
Début:
06 octobre 2022
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

On voit des PBX qui ne savent pas envoyer des callstop. On se retrouve avec des appels qui durent tous max_call_duration (2 heures).

Et lorsqu'un même numéro rappelle avant cette expiration, on pense que c'est un appel en cours et on ne fait quasi rien :

    def call_start(self, request, callee, caller, redirect=None, newtab=None, **kwargs):
        existing_call = Call.objects.filter(
            resource=self, callee=callee, caller=caller, end_timestamp=None
        ).last()
        if existing_call:
            existing_call.details = kwargs
            existing_call.save()
            response = {'data': existing_call.json()}

Conséquence, c'est comme s'il n'y avait pas de nouvel appel, rien n'est rafraîchi sur le portail agent. C'est vraiment visible dans mode test quand on essaye avec un ou deux numéros.

On devrait soit couper toutes les connexion existantes et en créer une nouvelle ou, plus simplement, mettre à jour le "start_timestamp" de l'appel de le plus récent.


Fichiers

Révisions associées

Révision 1d8bcee7 (diff)
Ajouté par Thomas Noël il y a plus d'un an

phonecalls: do not consider existing calls on call start (#69972)

Révision fa6df239 (diff)
Ajouté par Thomas Noël il y a plus d'un an

phonecalls: do not consider existing calls on call start (#69972)

Historique

#2

Mis à jour par Benjamin Dauvergne il y a plus d'un an

Plutôt que de changer start_date, ce qui perdra l'existence de l'appel précédent dans l'historique je proposerai plutôt de le fermer et le de réouvrir (et de ne pas fermer tous les autres appels, on ne sait jamais s'ils ont du double appel :) ).

#3

Mis à jour par Thomas Noël il y a plus d'un an

Benjamin Dauvergne a écrit :

Plutôt que de changer start_date, ce qui perdra l'existence de l'appel précédent dans l'historique je proposerai plutôt de le fermer et le de réouvrir (et de ne pas fermer tous les autres appels, on ne sait jamais s'ils ont du double appel :) ).

Oui, bonne idée, et après tout c'est plus en phase avec la réalité.

#4

Mis à jour par Thomas Noël il y a plus d'un an

Voilà une version simplifiée qui ajoute un nouvel appel quand il reçoit un callstart.

Je pense inutile toute intelligence, je n'imagine pas un PBX nous envoyer plusieurs callstart pour un même appel.

#5

Mis à jour par Benjamin Dauvergne il y a plus d'un an

  • Statut changé de Solution proposée à Solution validée

Thomas Noël a écrit :

Je pense inutile toute intelligence, je n'imagine pas un PBX nous envoyer plusieurs callstart pour un même appel.

Tu as toujours été un grand optimiste.

#6

Mis à jour par Thomas Noël il y a plus d'un an

  • Statut changé de Solution validée à Résolu (à déployer)
commit 1d8bcee7065fc850f8147a19e31d1cf4bc5310f2
Author: Thomas NOEL <tnoel@entrouvert.com>
Date:   Thu Oct 6 18:40:30 2022 +0200

    phonecalls: do not consider existing calls on call start (#69972)

#7

Mis à jour par Transition automatique il y a plus d'un an

  • Statut changé de Résolu (à déployer) à Solution déployée
#8

Mis à jour par Thomas Noël il y a plus d'un an

  • Sujet changé de phonecalls: sur un callstart d'un appel déjà existant, mettre à jour sa date de début à phonecalls: sur un callstart d'un appel déjà existant, créer un nouvel appel
#10

Mis à jour par Transition automatique il y a plus d'un an

Automatic expiration

Formats disponibles : Atom PDF