Development #48522
phonecalls : pouvoir indiquer une URL de redirect après notification de l'appel
0%
Description
Pour le cas où la notification des appels se passerait en front dans un navigateur il faudrait pouvoir rediriger l'agent vers sa page d'accueil téléphonique sur le portail-agent.
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Benjamin Dauvergne il y a plus de 3 ans
- Fichier 0001-phonecalls-allow-redirecting-to-portal-after-call-st.patch 0001-phonecalls-allow-redirecting-to-portal-after-call-st.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Benjamin Dauvergne il y a plus de 3 ans
- Fichier 0001-phonecalls-allow-redirecting-to-portal-after-call-st.patch 0001-phonecalls-allow-redirecting-to-portal-after-call-st.patch ajouté
La sempiternelle migration/pomponette.
Mis à jour par Nicolas Roche il y a plus de 3 ans
Je pige pas tout mais vu https://en.wikipedia.org/wiki/CDATA,
je dirais qu'il faut commenter comme ci-dessous pour l'utiliser dans une balise script :
<script>/*<![CDATA[*/window.open("https://agent-combo.dev.publik.love/allo/");/*]]>*/</script>
Mis à jour par Benjamin Dauvergne il y a plus de 3 ans
- Fichier 0001-phonecalls-allow-redirecting-to-portal-after-call-st.patch 0001-phonecalls-allow-redirecting-to-portal-after-call-st.patch ajouté
Ouaip ça ne sert à rien en fait. Mais en testant je ne m'aperçois que c'est bloqué par le blocker de popup, mais je suppose qu'ils sont déjà au courant à Toulouse.
Mis à jour par Benjamin Dauvergne il y a plus de 3 ans
Mis à jour par Thomas Noël il y a plus de 3 ans
Pourquoi ne pas carrément accepter un paramètre « redirect_url » dans l'URL ? Comme ça c'est totalement piloté par l'appelant (qui pilote déjà le "newtab" alors autant lui donner les clés du camion)
Mis à jour par Benjamin Dauvergne il y a plus de 3 ans
Thomas Noël a écrit :
Pourquoi ne pas carrément accepter un paramètre « redirect_url » dans l'URL ? Comme ça c'est totalement piloté par l'appelant (qui pilote déjà le "newtab" alors autant lui donner les clés du camion)
Pour pouvoir en changer sans avoir à traiter avec les gens qui gèrent Kiamo. Mais ça me va d'ajouter cette possibilité en plus. C'est un peu léger de toute façon de passer par des URLs ouvertes (même avec une API key elle sera publique dans le navigo des agents) comme ça pour poser un appel (je doute que les agents jouent à ça, mais n'importe qui pourra poser un appel sur le portail du collègue si il regarde un peu dans l'historique).
Mis à jour par Benjamin Dauvergne il y a plus de 3 ans
Mis à jour par Thomas Noël il y a plus de 3 ans
c'est bloqué par le blocker de popup, mais je suppose qu'ils sont déjà au courant à Toulouse.
On a eu confirmation de ça ?
Sinon, « pour pouvoir en changer sans avoir à traiter avec les gens qui gèrent Kiamo » tu as raison totalement, supprime ma mauvaise idée de permettre un redirect_url, désolé. Revenons à la proposition précédente, donc (note #7)à.
Mis à jour par Benjamin Dauvergne il y a plus de 3 ans
Thomas Noël a écrit :
c'est bloqué par le blocker de popup, mais je suppose qu'ils sont déjà au courant à Toulouse.
On a eu confirmation de ça ?
À moins d'avoir un navigateur qui ne bloque pas les popups, faire un window.open() qui ne découle pas d'un évènement utilisateur (clavier, souris, touchscreen) va déclencher le bloqueur de popup, tous les navagiteurs font ça aujourd'hui. Donc là avoir <script>window.open(...)</script> c'est forcément bloqué, partout, mais je vais ajouter un bouton et un message, normalement je dois pouvoir détecter la situation, car window.open(...) retourne null sur un bloquage.
Si vous voyez ce message c'est que votre navigateur bloque les popups, débloquez les. [ button "Continuer ]
Sinon, « pour pouvoir en changer sans avoir à traiter avec les gens qui gèrent Kiamo » tu as raison totalement, supprime ma mauvaise idée de permettre un redirect_url, désolé. Revenons à la proposition précédente, donc (note #7)à.
Ok.
Mis à jour par Thomas Noël il y a plus de 3 ans
Benjamin Dauvergne a écrit :
Thomas Noël a écrit :
c'est bloqué par le blocker de popup, mais je suppose qu'ils sont déjà au courant à Toulouse.
On a eu confirmation de ça ?
À moins d'avoir un navigateur qui ne bloque pas les popups, faire un window.open() qui ne découle pas d'un évènement utilisateur (...)
En fait je voulais juste savoir si on avait confirmation que Toulouse était au courant, connaissait le pb, etc... ou s'ils utilisent "autre chose" (aucune idée de ce que ça pourrait être).
Mis à jour par Benjamin Dauvergne il y a plus de 3 ans
- Fichier 0001-phonecalls-allow-redirecting-to-portal-after-call-st.patch 0001-phonecalls-allow-redirecting-to-portal-after-call-st.patch ajouté
Voilà, avec un contournement et un message d'explication.
Mis à jour par Thomas Noël il y a plus de 3 ans
Benjamin Dauvergne a écrit :
Voilà, avec un contournement et un message d'explication.
Je pense qu'on ne se comprend pas, mon idée c'est juste de poser la question à Toulouse. S'ils sont au courant du pépin (ie qu'ils l'ont déjà eu et réglé les navigos) alors inutile de compliquer le patch, je pense (c'est déjà assez bidouille comme ça ;)).
Mis à jour par Benjamin Dauvergne il y a plus de 3 ans
Thomas Noël a écrit :
Je pense qu'on ne se comprend pas, mon idée c'est juste de poser la question à Toulouse. S'ils sont au courant du pépin (ie qu'ils l'ont déjà eu et réglé les navigos) alors inutile de compliquer le patch, je pense (c'est déjà assez bidouille comme ça ;)).
Mon assurance à 100% que le problème a lieu ne te convainc pas ?
Mis à jour par Benjamin Dauvergne il y a plus de 3 ans
- Fichier 0001-phonecalls-allow-redirecting-to-portal-after-call-st.patch 0001-phonecalls-allow-redirecting-to-portal-after-call-st.patch ajouté
J'ai viré le code qui gère l'absence de popup, ça ira dans un autre ticket.
Mis à jour par Benjamin Dauvergne il y a plus de 3 ans
- Lié à Development #48575: phonecalls : gérer le bloquage des popups avec l'option newtab=1 ajouté
Mis à jour par Thomas Noël il y a plus de 3 ans
- Statut changé de Solution proposée à Solution validée
Benjamin Dauvergne a écrit :
J'ai viré le code qui gère l'absence de popup, ça ira dans un autre ticket.
Cette partie :
redirect_url = self.redirect_url
# redirect to agent's portal
if redirect and redirect_url:
return HttpResponseRedirect(redirect_url)
# open agent's portal in a new tab
if newtab and redirect_url:
return render(request, 'phonecalls/start_call_newtab.html', {'redirect_url': redirect_url})
return response
je l'aurai vue ainsi (qui cache moins le "return response") :
if self.redirect_url:
if redirect:
# redirect to agent's portal
response = HttpResponseRedirect(redirect_url)
elif newtab:
# open agent's portal in a new tab
response = render(request, 'phonecalls/start_call_newtab.html', {'redirect_url': redirect_url})
return response
mais c'est la même chose donc ack.
Mis à jour par Benjamin Dauvergne il y a plus de 3 ans
- Fichier 0001-phonecalls-allow-redirecting-to-portal-after-call-st.patch 0001-phonecalls-allow-redirecting-to-portal-after-call-st.patch ajouté
- Statut changé de Solution validée à Solution proposée
Mis à jour par Benjamin Dauvergne il y a plus de 3 ans
- Statut changé de Solution proposée à Résolu (à déployer)
commit faa52f04194f7385e03dc8c4eec1a9b83357bea9
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date: Fri Nov 13 12:16:42 2020 +0100
phonecalls: allow redirecting to portal after call-start (#48522)Mis à jour par Benjamin Dauvergne il y a plus de 3 ans
Thomas Noël a écrit :
Lapin compris.
Je faisais autre chose, j'ai glissé :)
Mis à jour par Frédéric Péters il y a plus de 3 ans
- Statut changé de Résolu (à déployer) à Solution déployée
phonecalls: allow redirecting to portal after call-start (#48522)