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

mejora modulo call center, se puede configurar diferentes callerid uno por campaña para mostrar el numero deseado segun la campaña #37

Merged
merged 12 commits into from
Feb 21, 2019

Conversation

hgmnetwork
Copy link
Contributor

@hgmnetwork hgmnetwork commented Feb 19, 2019

mejora al crear campañas o editarlas la opción para poder mostrar un callerid en cada campaña o bien en blanco mostraría el que tenga el troncal por defecto. También almacena en cada llamada el callerid indicado para poder saber el numero que se mostró en cada llamada.

cambios para mostrar caller id por campaña segun se configure a tiempo real y se almacene
modificado sql para las llamadas con caller id
añadimos el campo de callerid para crear/modificar cada campaña
añadimos el texto en ingles por defecto para mostrar diferente caller id por campañas
añadimos textos para mostrar caller id diferente segun campaña
añadimos al modulo call center la opción de añadir callerid diferente a cada campaña, ideal para mostrar diferentes numeros salientes según la campaña saliente que sea, o por defecto muestra el troncal (actualmente todas asi )
añadido el que obtenga el caller id de la campaña
@asternic
Copy link
Contributor

Muchas gracias! Mucho mejor! Se ve muy prolijo, noto solo dos pequeñas cositas:

  1. Falta modificar setup/installer.php para hacer un ALTER TABLE de las tablas a las que se le ha agregado el campo callerid. Sin esto, los usuarios que hagan upgrade del módulo se les romperá pues sus tablas no tendrás en campo callerid agregado.

  2. Faltaría también hacer los cambios en las definiciones de create table en setup/firstboot_call_center.sql

  3. No es consistente la definición de campo callcenter en las tablas, en una tabla lo defines como int(10) y en otra como int(12), habría que unificar y que tengan la misma definición. Y siendo que en varias otras tablas dicho campo ya existe y es un varchar(15), lo correcto sería definir el campo en las tablas en que los agregas como varchar(15) también, de modo que sea consistente en todas las tablas.

Saludos,

@hgmnetwork
Copy link
Contributor Author

hgmnetwork commented Feb 19, 2019 via email

añadimos a la tabla calls y campaign el callerid como varchar 15 hgmnetwork.com 20-02-2019
@hgmnetwork
Copy link
Contributor Author

Hola, he modificado los parámetros que me indicaste y creo que ahora estaría todo correcto, no estoy seguro si hace falta algo más. si esta ok, en cuanto se añada subo otros cambios para ir de uno en uno 😄

@hgmnetwork
Copy link
Contributor Author

Hola asternic, has podido ver si esta ahora todo correcto ? cualquier cosa me avisas. para poder enviar la siguiente mejora

@asternic
Copy link
Contributor

Hola Moises, dame un tiempo para revisarlo mejor, a primera mirada se ve bien. Supongo que al fin del día ya podríamos aceptarlo. Muchas gracias por todo.

@asternic asternic merged commit d1ed867 into IssabelFoundation:master Feb 21, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants