IssabelFoundation / callcenter

Community Call Center Module using ECCP Protocol.
38 stars 62 forks source link

Create PaloSantoBreaks.class.php #12

Closed hgmnetwork closed 5 years ago

hgmnetwork commented 6 years ago

añadida la funcion ver agendadas para mostrar las llamadas agendadas de las campañas casoft y hgmnetwork.com

asternic commented 5 years ago

Hola,

Te pido por favor revises algunas cosas para poder aceptar todas tus colaboraciones/pull requests:

1) Por favor trata de no reutilizar nombres de archivos o clases ya existentes, como en este caso, sobre todo si la clase cumple una función distinta a la clase usada como modelo. No podemos aceptar esta modificación para crear el archivo PaloSantoBreaks.class.php, por la sencilla razón de que dicha clase ya existe en el paquete. Del mismo modo, si la idea fue usar dicha clase como modelo para listar llamados agendados, deberías pensar en renombrar no solo el archivo sino la clase misma de modo que indique su función, en lugar de dejal mismo nombre de la clase usada como modelo. Hacer una limpieza de todo comentario que no ataña, o variables que no se usen, sería un plus también, o bien pensar en renombrar variables... que caso tiene una variable $idbreak en una clase que muestra llamados agendados? y que además no se usa?

2) Otro punto muy importante, por favor trata de hacer UN SOLO pull request por cada funcionalidad o cambio, y no por cada archivo modificado. Si agregar la lista de llamados agendados comprende la creación de una clase nueva, y modificar 4 archivos existentes o los que fuesen, debes entonces generar UN SOLO pull requests con todas esas modificaciones para esa funcionalidad. Hacer un pull request separado por cada archivo modificado nos hace demasiado complicado tratar de replicar el cambio, o de hacer un rollback de manera apropiada si hubiera problemas.

Hoy hemos aceptado los pull requests de otros usuarios que cumplían estas condiciones, lo mismo podremos hacer con todas tus contribuciones si se ordenan y emprolijan un poco más.

Muchas gracias por tu colaboración.