Closed hgmnetwork closed 5 years ago
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,
Genial ya voy entendiendo mejor esto. Mañana hago los cambios que me pides y lo vuelvo a enviar.
También enviaré otras mejoras.
Un Cordial saludo Moisés Hdez. www.hgmnetwork.com tel. 900 87 88 85
El mar., 19 feb. 2019 22:27, Nicolas notifications@github.com escribió:
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,
— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/IssabelFoundation/callcenter/pull/37#issuecomment-465336912, or mute the thread https://github.com/notifications/unsubscribe-auth/Ac4piGGpfP8KlTfhCEs8B0bLQSu0yuw0ks5vPHpggaJpZM4bD87d .
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 😄
Hola asternic, has podido ver si esta ahora todo correcto ? cualquier cosa me avisas. para poder enviar la siguiente mejora
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.
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.