GovernIB / rolsac

Gestor de continguts administratius
4 stars 1 forks source link

Implementació del flux de control previ a la publicació #486

Closed bgarcia-dgtic closed 2 years ago

bgarcia-dgtic commented 3 years ago

Es sol·licit la implementació del flux de control previ a la publicació de procediments serveis.

slromero-indra commented 3 years ago

Buenas, hemos terminado con una primera versión (falta el tema de envío de emails que estamos en ello). Los cambios están en la rama rolsac-1.10 (hay scripts que ejecutar que crean la tabla de mensajes y el campo pendiente de validar en procedimiento y servicios)

bgarcia-dgtic commented 3 years ago

Hola, He descarregat les darreres modificacions, he executat l'escript rolsac_update_schema_from1.9.1_to_1.10.0.sql sense problemes i he afegit les properties. He desplegat l'aplicació en local i m'ha funcionat, però quan entro dins procediments o serveis i espitjo sobre els missatges, m'apareixen errors als logs. Us els envio per correu. També he intentat desplegar l'aplicació a DEV però dóna error. Us envio els logs per correu.

slromero-indra commented 3 years ago

Buenas, hemos respondido por email, podéis validar que los cambios funcionan y que no surge ningún problema?

bgarcia-dgtic commented 3 years ago

Hola, encara dóna algún error. Us he enviat un correu.

slromero-indra commented 3 years ago

Ya está subido el cambio, podéis verificarlo?

bgarcia-dgtic commented 3 years ago

He desplegat els nous canvis i no ha donat error referent a l'hora d'obrir un procediment/servei. Ara farem proves per veure si funciona correctament el flux de control

bgarcia-dgtic commented 3 years ago

Hola, A l'hora d'eliminar un procediment/servei dóna error. Us he enviat un correu amb els logs.

bgarcia-dgtic commented 3 years ago

Bon dia, Fent més proves hem detectat el següent:

episano-dgtic commented 3 years ago

Comentarios de la OAE que se añaden a los comentarios anteriores: COMUNES: a) Cambiar etiqueta "Pendent validar" por "Pendent validar/revisar" tanto en pantalla procedimientos/servicios como filtro buscador b) Revisar los textos de los desplegables tanto el backoffice castellano y catalán, en todos los filtros. En las opciones "nuevas" están idioma intercambiado y en checks antiguos hay un batiburrido en algunos casos. Si no se pudiera elegir el idioma de las opciones por defecto sería la catalana. c) No hay ningún feedback automático específico si un supervisor publica un procedimiento tras petición. Incluir un mensaje específico que se ha procedido a publicar el procedimiento/servicio. Otra opción sería incluir el estado del procedimiento servicio a todos los mensajes junto al usuario creó mensaje/fecha leido. d) La ventana de mensajería de los procedimientos/servicios sea desplazable por la ventana.

GESTOR: a) En el check "Missatges per llegir" únicamente tenga la opción "sin mensajes pendientes" y "mensajes pendientes gestor", para simplificar los resultados de dicho perfil, y únicamente pueda filtrar lo que tenga pendiente para su perfil. b) En procedimiento/servicio en estado interno no debería aparecer la opción "tancar". c) No permitir enviar a publicar hasta que no esté completo, es decir tenga todos los campos obligatorios para su publicación (LOPD, noramtiva, Trámites, materias...)

Si alguno de los puntos requiere muchas horas, indicarlo para cumplir plazos marcados.

slromero-indra commented 3 years ago

Los cambios pedidos por bgarcia-dgtic ya están comiteados.

Respecto a los cambios propuestos por Estela, los revisamos y os decimos algo en la mayor brevedad posible.

slromero-indra commented 3 years ago

Respecto a la OIE.

En el apartado de comunes:

En el apartado de gestor:

episano-dgtic commented 3 years ago

okey sergio. De la opción comunes, descarta la petición d, le haremos saber requiere consumo horas extra. Respecto a la opción c) Sabemos que se genera un mensaje al revisarlo pero ello no implica que se haya publicado el procedimiento. Y tal y como está concebida la aplicación, no podemos saber cuando se ha publicado un procedimiento, ni durante cuanto tiempo ha estado publicado, ya que no hay un histórico del tema. Si se incluyera el estado del procedimiento en todos los mensajes tendríamos esta información. Si es complicado, al menos lo que sugieres tú, y que aparezca un mensaje especificando se ha publicado.

slromero-indra commented 3 years ago

Hemos subido los últimos cambios. Se ha creado unas nuevas propiedades en el jboss-service.xml:

   <!--  Supervisor revisado de proc/servs  -->
    es.caib.rolsac.procServ.cambioValidacion.es=Se ha actualizado del estado {0} a {1}
    es.caib.rolsac.procServ.cambioValidacion.ca=S'ha actualizat de l'estat {0} a {1}

Este mensaje se utilizará cuando el supervisor cambie el estado del proc/serv. Os pasamos una captura ejemplo (un mensaje es cuando sólo se cambia el estado mientras que el otro literal es cuando se cambia el estado y además se quita el check de pendiente validar):

imatge

Con estos cambios, sólo falta el detalle de poder enviar email desde el perfil supervisor (estamos teniendo problemas con la librería que es bastante antigua, en cuanto lo tengamos, os avisamos).

bgarcia-dgtic commented 3 years ago

Bon dia, Hem pujat els canvis i ens hem adonat que quan es marca un proc/serv per publicar, no s'activa el check de pendent de revisar. No obstant sí que es genera el missatge "Se sol·licita la publicació del contingut".

bgarcia-dgtic commented 3 years ago

Bon dia, També ens sol·liciten a veure si és possible que els asteriscs blaus i taronges dels missatges siguin un poc més grans.

slromero-indra commented 3 years ago

Hemos realizado más grande los asteríscos para cuando hay mensajes pendientes para leer para el gestor o/y supervisor.

Respecto al problema comentado ayer , cuál es la casuística para replicarlo? Es estando con un usuario gestor y guardando un proc/serv en estado interno y marcando la acción publicar?

Hemos introducido la opción de enviar emails cuando se tiene permisos de supervisor. Se ha procedido a crear una tabla donde se guardarán los mensajes y una tarea que se encargará de enviar emails pendientes y otra tarea para borrar los que lleven una semana en la tabla.

La nueva tabla a ejecutar es (está en el sql para la 1.10):

CREATE TABLE RSC_MSJMAI (
  MAI_CODI NUMBER(19,0),
  MAI_PROSRV NUMBER(19,0),
  MAI_FROM VARCHAR2(100 CHAR),
  MAI_TO  VARCHAR2(100 CHAR),
  MAI_TITUL VARCHAR2(100 CHAR),
  MAI_CONTE  VARCHAR2(500 CHAR),
  MAI_ERROR  VARCHAR2(500 CHAR),
  MAI_TIPO  VARCHAR2(3 CHAR),
  MAI_FECCRE DATE,
  MAI_FECENV DATE,
  MAI_ENVIAD NUMBER(1,0) DEFAULT 0,
  CONSTRAINT RSC_MSJMAI_PK PRIMARY KEY (MAI_CODI)

);

create sequence RSC_SEQ_MAI;

Hay que darle permisos a la tabla y columna al usuario que utilice el jboss (supongo que www_rolsac) y crearles los respectivos sinónimos. Ejemplos:


#Los permisos a ejecutar con ROLSAC
grant select, insert, update, delete  on RSC_MSJMAI to www_rolsac;
grant select  on RSC_SEQ_MAI to www_rolsac;

#Los sinonimos a ejecutar con WWW_ROLSAC
CREATE  SYNONYM RSC_MSJMAI FOR ROLSAC.RSC_MSJMAI;
CREATE  SYNONYM RSC_SEQ_MAI FOR ROLSAC.RSC_SEQ_MAI;

También hay que tocar el jndi del smtp (rolsac-mail-service.xml , está en el deploycaib del jboss) cuyo jdni debe ser java:/es.caib.rolsac.mail. Os paso una captura de como lo tenemos en nuestro entorno (puede que tengáis que añadir algo más como el puerto si no es el estándar el smtp de caib):

imagen

Además, tenéis que añadir lo siguientes parámetros en el jboss-service para configurarlo (los 3 primeros son propiedades de test que no hace falta configurar, los dos siguientes son las tareas cronn para enviar/limpiar emails, mientras que los siguientes son para el título y el contenido del envío de emails):

<!--  Envio de emails  -->
    es.caib.rolsac.procServ.email.test=N
    es.caib.rolsac.procServ.email.test.from=emailFrom@caib.es
    es.caib.rolsac.procServ.email.test.to=emailTo@caib.es
    es.caib.rolsac.procServ.email.taskEnvio=0 0/10 * * * ?
        es.caib.rolsac.procServ.email.taskLimpieza=0 0 1 * * ?
        es.caib.rolsac.procServ.email.srv.titulo=Actualizacion en el servicio {0}
        es.caib.rolsac.procServ.email.srv.contenido=El supervisor {0} comenta que:\n {1} \n Mas informacion en la url http://www.caib.es/rolsacback/catelegServeis/catalegServeis.do?itemId={2}
    es.caib.rolsac.procServ.email.proc.titulo=Actualizacion en el procedimiento {0}
        es.caib.rolsac.procServ.email.proc.contenido=El supervisor {0} comenta que:\n {1} \n Mas informacion en la url http://www.caib.es/rolsacback/catalegProcediments/catalegProcediments.do?itemId={2}
bgarcia-dgtic commented 2 years ago

Bon dia, Referent a la problemàtica a l'hora de voler publicar un proc/ser, efectivament és dóna quan un gestor guarda un proc/ser en estat intern i el marca l'acció publicar.

bgarcia-dgtic commented 2 years ago

Hola, Hem fet proves d'enviaments d'emails però no acaba de funcionar. La primera prova que hem fet ha funcionat, és a dir, un gestor ha creat un missatge i el supervisor l'hi ha enviat un correu correctament. Després hem tornat a fet una prova i ja no ha anat bé, el supervisor ha enviat email però el gestor no l'ha rebut. Mirant sa base de dades, hem vist que el correu destinatari és null@caib.es. Captura1

Als logs hem vist el següent: Captura2

Després hem fet més proves d'enviar correu, però tampoc ha funcionat. Ni tan sols s'emmagatzema a la BBDD.

També hem comprovat que el text de l'email que s'envia al gestor quan es tracta d'un servei no és correcte, és a dir, el text que posa fa referencia a procediments en lloc de serveis.

slromero-indra commented 2 years ago

Ya están resueltos todos los problemas (incluido el marcar con el check pendiente validar cuando se pide publicar).

Respecto al email, ya no almacena el email null. Hay que tener en cuenta que el email se envía al último gestor que haya realizado un mensaje. El null se obtenía porque no había ningún mensaje de un usuario gestor. Ahora, si no hay ningún gestor, se lanza un error avisando del problema y pidiendo que al usuario que desseleccione la opción de enviar email.

Para poder validar los datos, mejor borrar antes los mensajes ya creados con la siguiente sql:

delete from rsc_msjmai;

bgarcia-dgtic commented 2 years ago

Hola, Hem descarregat les darreres modificacions i pareix que el control de flux funciona correctament. No obstant, hem detectat deficiències en els emails que s'envien:

  1. En els emails dels serveis, la url no és correcte, no substitueix {2} pel número de servei.
  2. Hi caràcters especials que no s'envien. Per exemple, si volem enviar un mail amb el text "publicado con episano@çaib.es la c es especial. áéíóú vocales con acentos. ñ eñe.", realment s'envia el següent: "publicado con episano@aib.es la c es especial. vocales con acentos. ee.".

També ens hem adonat que si un procediment està en estat "Reserva", permet eliminar tràmits. No deixa crear tràmits ni modificar-ne el seu contingut, però sí eliminar tràmits (no hauria de deixar eliminar). Respecte a la resta de mòduls laterals, tant dels procediments com dels serveis, no permet la seva modificació (això és correcte).

slromero-indra commented 2 years ago

Hemos comiteado los cambios:

Hemos probado enviando el siguiente texto y se envía correctamente:

 Prueba de mensajes:
áéíóÚ àèìòÙ äëïöÜ ñ y ç trencada 

Respecto al tema de un procedimiento en estado de reserva. Hemos comprobado y, aunque se puede pulsar el icono de quitar trámite, no se puede guardar el cambio y no se puede borrar.

bgarcia-dgtic commented 2 years ago

Hola, Referent als caràcters especials i la url del servei, funciona correctament. Referent al tràmit, efectivament si es prem sobre la creueta del tràmit, s'elimina però no deixa guardar el procediment, per tant realment el tràmit no s'elimina. No obstant, si s'entre al tràmit, apareix el botó "Eliminar" i si es prem sí que deixa eliminar-lo. No hauria d'aparèixer el botó "Eliminar".

slromero-indra commented 2 years ago

Hemos realizado pruebas y sólo nos aparece el botón de eliminar un trámite con usuario supervisor. Cuando está en estado reserva, no se puede eliminar el trámite ni el usuario supervisor ni el usuario gestor?

Captura de trámite como usuario gestor imatge

Captura de trámite como usuario supervisor: imatge

slromero-indra commented 2 years ago

Respecto al problema con el botón, está correcto lo que os hemos indicado?

bgarcia-dgtic commented 2 years ago

Hola, Referent al funcionament del botó Eliminar, quan el procediment està en estat "Reserva", només el supervisor podrà eliminar. El gestor no hauria de fer cap modificació. A nosaltres sí que ens apareix el botó "Eliminar" d'un tràmit quan es tracta d'un gestor. Adjunto captura: imatge

slromero-indra commented 2 years ago

Hemos actualizado el js de trámites para ocultar el botón de elimina.

bgarcia-dgtic commented 2 years ago

Hola, A descarregat les noves modificacions però encara es mostra el botó "Elimina". imatge

slromero-indra commented 2 years ago

Subido de nuevo el cambio. El problema se producía cuando era estado interno pero no tenía el check de pendiente validación activo.

Se ha actualizado para que la aparición del botón de eliminar siga la misma casuística que al cargar el detalle de procedimientos.

bgarcia-dgtic commented 2 years ago

Hola, Funciona correctament

slromero-indra commented 2 years ago

Tras lo comentado en la reunión, se va a proceder a realizar los siguientes cambios:

  1. Añadir en los mensajes código usuario + nombre y apellidos
  2. Cuando un supervisor guarde un proc/servicio que está en estado interno y mantiene el estado interno y desmarca el check de validar (es decir, que lo ha validado pero fallan cosas y no se puede publicar) debe salir un popup para elegir si envía o no un email con un texto fijo (el texto se configurará por propiedades del sistema)
bgarcia-dgtic commented 2 years ago

Hola, S'ha de fer una modificació més, s'ha de canviar un literal. Al desplegable "Acció", de l'estat "Interna", s'ha de substituir "Publicar" per "Validar" ja que el gestor mai enviarà a publicar sinó a validar.

slromero-indra commented 2 years ago

Cuando podáis, nos validáis los cambios. Las nuevas propiedades a añadir son las siguientes:

es.caib.rolsac.procServ.mantieneEstadoInterna.es=Se ha mantenido el estado interna
es.caib.rolsac.procServ.mantieneEstadoInterna.ca=S'ha mantingut l'estat interna
bgarcia-dgtic commented 2 years ago

Bon dia,

Després d'haver pujat els canvis a DEV i fer proves hem detectat el següent:

slromero-indra commented 2 years ago

Hemos actualizado la aplicación con los cambios para:

bgarcia-dgtic commented 2 years ago

Bon dia, Pareix que ja no es dupliquen els missatges a l'historial de missatges.

bgarcia-dgtic commented 2 years ago

No obstant, hem detectat que quan un procediment/servei té el títol llarg, no s'envia email al gestor. Us he enviat un correu amb logs. S'hauria d'augmentar la mida del títol dels emails a 300 caràcters.

slromero-indra commented 2 years ago

Hemos actualizado el tamaño del campo del titulo a 356 (256 es el tamaño del titulo de procedimientos y servicios, y hemos sumado 100 caracteres más debido a que se concatena con un literal de las propiedades de rolsac). La sql:

ALTER TABLE RSC_MSJMAI MODIFY MAI_TITUL VARCHAR2(356 CHAR);

Además de ejecutar el comando, hay que actualizar el ear de rolsac, porque en el fichero hbm de mensajes, estaba especificado el máximo a 100 (ahora 356).

episano-dgtic commented 2 years ago

Nuevos cambios solicitados:

1) El desplegable de acciones para el perfil GESTOR debe tener en cuenta el estado del procedimiento. De modo que si está interno aparezcan las opciones(desar, validar) y en estado público (republicar, eliminar, tancar). La idea es ya que en estado interno el gestor puede eliminar el procedimiento él mismo, no remita la petición eliminar a los supervisores.

2) Analizar la mejora de exportación o guardar el contenido del pop-up MENSAJES. Tal vez la opción más sencilla es un botón que provoque la copia de todo el contenido pop-up en el portapapeles.

3) Respecto mensajes de e-mail se desea que el correo automático enviado al gestor tras la revisión supervisor manteniendosé a interno incluya tb. la url de acceso al procedimiento/servicio para l backoffice tal y como ocurre en los e-mails manuales del pop-up. imatge Més informació a la url https://dev.caib.es/rolsacback/catalegServeis..

4) En los emails remitidos manualmente desde la consola de pop-up, se desea que se identifique al usuario no sólo por código sino tb. con sus datos personales. u187980 Bartolomé Garcia imatge

episano-dgtic commented 2 years ago

El primer pto solicitado (desplegable de acciones), es limitante para el paso a producción programada para el 13 de diciembre.

slromero-indra commented 2 years ago

Buenas, respecto al primer punto, no entendemos que se desea, actualmente las acciones son las siguientes:

En estado interna: imagen

En estado publicado: imagen

¿Se desea quitar la acción eliminar y mostrar el botón de eliminar cuando se sea gestor y el estado sea interna?

slromero-indra commented 2 years ago

Buenas, hemos subido una primera versión del punto 2, 3 y 4.

Del punto 1 no tenemos claro si lo que se desea es que el botón eliminar tiene que salir cuando es usuario gestor y estado interno (desaparecería la acción de eliminar)

Respecto al punto 2, realiza una copia al portapapeles del contenido de texto. Copia tanto los mensajes como la info de fechas/usuarios. Debería copiarse sólo el texto? O debería de guardarse en formato html por si se quiere tener un formato para enviar por email? Sólo los mensajes incluidos o también las fechas/usuarios?

Respecto al punto 4, hay que añadir lo siguiente por propiedades que es la url a la que debe redirigir:

        es.caib.rolsac.procServ.proc.url=http://www.caib.es/rolsacback/catalegProcediments/catalegProcediments.do?itemId={0}
        es.caib.rolsac.procServ.serv.url=http://www.caib.es/rolsacback/catelegServeis/catalegServeis.do?itemId={0}
episano-dgtic commented 2 years ago

REspecto al pto 1, efectivamente hay que quitar en el desplegable la opción eliminar porque ya existe el botón eliminar para que lo realicen los propios gestores. interno

Consulto las posibilidades de la exportació datos pop-up para que aseguren que es lo que quieren exactamente.

slromero-indra commented 2 years ago

Hemos subido los últimos cambios para el punto 1, estando ya pendiente de validar todos los puntos (a excepción del punto 2 cuando sepamos que desean, pero ya realiza un copiar al portapapeles de toda la información).

bgarcia-dgtic commented 2 years ago

Hola, Pareix que està correcte, però hem detectat que als procediments s'han descol·locats els botons. imatge

slromero-indra commented 2 years ago

Buenas, hemos subido el cambio para el tema del botón, podéis validarlo?

Sólo quedaría detallar que datos desean copiar del punto 2.

bgarcia-dgtic commented 2 years ago

En relació a la desconfiguració del botó, funciona correctament. Pel que fa al punt 2, de moment ho deixam així tal qual està.

episano-dgtic commented 2 years ago

Otra mejora: Actualmente Si se desactiva el check validación.. se incluye un mensaje "S’ha revisat la informació"

Ahora desean que: Si se desactiva el check validación y el estado se guarda como interno el mensaje automático sea: "S’ha revisat la informació i s’han detectat incorreccions a corregir" y si se desactiva el check validación y el estado está público el mensaje sea "S’ha revisat la informació".

slromero-indra commented 2 years ago

Hemos subido el cambio para cambiar el mensaje automático cuando se quita el check de validación y sigue en estado interna. Hay que incorporar los siguientes cambios en el fichero de propiedades:

 es.caib.rolsac.procServ.revisadoConErrores.es=Se ha revisado la informacion i se han detectado incorrecciones a corregir
 es.caib.rolsac.procServ.revisadoConErrores.ca=S'ha revisat la informacio i s'han detectat incorreccions a corregir
episano-dgtic commented 2 years ago

Se ha visto afectado el módulo de los documentos relacionados de las fichas. No se permite incluir /modificar ni eliminar los documentos relacionados en fichas.

episano-dgtic commented 2 years ago

tb. se ha visto afectado la gestión documentos relacionados de las normativas.