pengovbr / mod-sei-pen

Módulo de Integração ao Tramita.GOV.BR
30 stars 23 forks source link

Melhoria da Mensagem de Alteração na Ordem de Documentos do Processo - Indicar ordem #261

Open guilhermeadc opened 1 year ago

guilhermeadc commented 1 year ago

Sua solicitação de melhoria está relacionada a um problema? Por favor descreva. Uma descrição clara e concisa de qual é o problema. Ex. Sempre fico frustrado quando [...]


Descreva a solução que você gostaria Melhorar a mensagem apresentada ao usuário nos casos de mudança de ordem para indicar o documento e qual foi a mudança aplicado, conforme descrito abaixo:

Apresentar mensagem atual e detalhar com as informações abaixo: "A ordem do documento XXXXX foi modificada na árvore do processo, mudando da posição 99 para a posição 55¨.


Descreva alternativas que você considerou Uma descrição clara e concisa de quaisquer soluções ou recursos alternativos que você considerou.


Contexto adicional Desenvolvimento deverá ser uma complementação do Pull Request #202 em que o documento e a mudança da ordem deverá ser informada ao usuário.

euviniciusmendonca commented 1 year ago

"Não foi possível enviar o processo <Nº do Processo> por meio do Tramita.GOV.BR, em decorrência de alteração da ordem de um ou mais documentos na árvore do processo.

A seguir, a lista dos documentos com ordem alterada:

A ordem do documento XXXXX foi modificada na árvore do processo, mudando da posição 99 para a posição 55. A ordem do documento XXXXX foi modificada na árvore do processo, mudando da posição 99 para a posição 55. . . . .

Sugere-se desfazer as alterações acima listadas antes de realizar nova tentativa de trâmite.

Mantenha sempre a ordem original dos documentos de processos recebidos pelo Tramita.GOV.BR, uma vez que sua instrução foi realizada por outro órgão. Tenha em mente que qualquer alteração nessa ordem pode impedir um novo trâmite do processo."

euviniciusmendonca commented 1 year ago

Solicita-se o vídeo como evidência da melhoria !!!

michael-ferreira2022 commented 1 year ago

https://drive.google.com/file/d/1Xgnm9t10SE5Tb4sAVeP4P6fBjHRY78HT/view?usp=share_link

euviniciusmendonca commented 1 year ago

Tem como colocar o texto na íntegra aqui? a qualidade do vídeo não possibilita a leitura do texto.

michael-ferreira2022 commented 1 year ago

Inconsistência identificada no documento de ordem '1' do processo tramitado por este NRE, '0000021166752023', com protocolo '99990.000001/2023-19'. Não foi possível enviar o processo '99990.000001/2023-19' por meio do Tramita.GOV.BR, em decorrência de alteração da ordem de um ou mais documentos na árvore do processo. A seguir, a lista de documentos com ordem alterada: A ordem do documento 000002 foi modificada na árvore do processo, mudando da posição 1 para a posição 2. A ordem do documento 000003 foi modificada na árvore do processo, mudando da posição 2 para a posição 1. Sugere-se desfazer as alterações acima listadas antes de realizar nova tentativa de trâmite. Mantenha sempre a ordem original dos documentos de processos recebidos pelo Tramita.GOV.BR, uma vez que sua instrução foi realizada por outro órgão. Tenha em mente que qualquer alteração nessa ordem pode impedir um novo trâmite do processo.

euviniciusmendonca commented 1 year ago

@caduvieira de acordo com a mensagem proposta e com a solução proposta para a issue.

davidnascimento1618 commented 1 year ago

@euviniciusmendonca, seria mais viável e menos oneroso, a inclusão de um botão de reordenação na tela de ordenação da árvore do processo, ou seja, ao invés de mostrar mensagem por mensagem para o usuário, para que ele fosse realizando os ajustes de posicionamentos íntegros dos documentos dentro do processo recebido, exibiria simplesmente uma mensagem genérica informando que a ordem foi alterada e, ao mesmo tempo, orientando-o à dirigir para a tela 'Ordenar Árvore do Processo', onde nesta, iria ter um botão/ícone, cuja função, seria reordenar a lista de documentos no processo, conforme a ordem original. O fato é que evitaria que a mensagem ficasse muito grande para o usuário (mensagem sendo exibida completa) ou que fosse mostrada por partes (várias etapas, em cascata, até que chegasse de fato a ordem original).

O ícone de 'Reordenar' ficaria, por exemplo, abaixo das setas de mover para cima e para baixo, conforme você pode vê na ilustração abaixo:

image

O que você acha? Poderemos implementar desta forma?

davidnascimento1618 commented 1 year ago

@euviniciusmendonca, poderia descrever a mensagem genérica que deseja, no que foi discutido e acertado no comentário acima?

euviniciusmendonca commented 1 year ago

A minha proposta é a seguinte:

Houve uma alteração na ordem dos documentos no processo, o que impede o reenvio de um processo que já foi tramitado pela plataforma. Portanto, é recomendado reordenar os documentos de acordo com a ordem original.

Caso você seja um usuário sem permissão para reordenar o processo, é necessário entrar em contato internamente para identificar quem possui essa permissão.

davidnascimento1618 commented 1 year ago

Caro @michael-ferreira2022, após reteste, foi verificado que a mensagem encontra-se atualizada, no entanto, o ícone de 'Reordenar', que ficaria na tela de 'Ordenar Árvore do Processo', não foi encontrado, conforme evidência abaixo:

Ícone de reordenação da árvore de processo não encontrado

Favor, atualizar o pull request com a inclusão do ícone de reordenação de documentos, funcional e, assim que realizar os testes, incluir uma evidência (em vídeo ou .gif) com a melhoria em questão. Grato desde já.

michael-ferreira2022 commented 1 year ago

Bom dia @davidnascimento1618 . O botão para reordenação dos documento encontra-se neste PR https://github.com/supergovbr/super/pull/301.

davidnascimento1618 commented 1 year ago

Prezado, @michael-ferreira2022, verifiquei no PR em questão, que o ícone de reordenação não condiz com a funcionalidade solicitada, foi observado que foi utilizado o mesmo ícone de deleção, ou seja, a "lixeirinha", segue abaixo (em arquivo .svg) o ícone correto que deverá ser atualizado. Grato desde já.

ícone_reordenar_documentos

michael-ferreira2022 commented 1 year ago

SEI - Reordenar.webm

davidnascimento1618 commented 1 year ago

Prezado, @euviniciusmendonca, conforme teste funcional realizado, a solicitação de melhoria encontra-se implementada segundo o requisito descrito.

OBS.: Como foi necessário alterar o core do SEI e a estratégia do projeto SUPER foi modificada junto ao TRF4, para que a funcionalidade em questão funcione, terão que ser atualizados as versões do SEI e do módulo do Tramita.

davidnascimento1618 commented 1 year ago

Novos testes foram realizados e foi verificado que no seguinte cenário o sistema está apresentando falha no comportamento:

  1. Org-1 inclui 3 documentos ao processo e, depois altera a ordem destes, modificando o documento posicionado na ordem 3 para a ordem 1;
  2. Org-1 envia o processo para o Org-2;
  3. Org-2 inclui mais dois documentos ao referido processo e, envia-o para o Org-1;
  4. Org-1 altera a ordem destes novos documentos incluídos pelo Org-2, modificando o documento posicionado na ordem 5 para a ordem 4 (ou seja, apenas houve uma troca do último para o penúltimo);
  5. Org-1 tenta enviar o processo para o Org-2 (obviamente, o sistema vai apresentar uma mensagem impedindo o envio, porque houve uma alteração nos documentos incluídos pelo Org-2);
  6. Org-1 reordena os documentos (utilizando a nova funcionalidade) e envia novamente o processo para o Org-2;
  7. Sistema vai apresentar uma mensagem impedindo o envio, porque no momento da reordenação, houve uma alteração na ordem dos documentos, no entanto não deveria.

Abaixo seguem 3 vídeos evidenciando o problema:

Steps 1 e 2:

https://github.com/pengovbr/mod-sei-pen/assets/87980947/d70b5f2c-5a95-4260-bd92-7da8e63278e8

Step 3:

https://github.com/pengovbr/mod-sei-pen/assets/87980947/96aa5702-7dee-4b57-862b-f0c40549009e

Steps 4, 5, 6 e 7:

https://github.com/pengovbr/mod-sei-pen/assets/87980947/171078af-c73a-4255-86fc-d374669f987a

michael-ferreira2022 commented 1 year ago

Necessário verificar o fluxo atual do alterar ordem do documento.

michael-ferreira2022 commented 12 months ago

Ao tentar executar o passo a passo acontece este erro no passo III. Mesmo aguardando um período longo ao realizar a tentativa de enviar o processo de volta para a unidade 1 o erro persiste. Também tentei gerar outros envios, mas todos tiveram este mesmo problema.

Image

davidnascimento1618 commented 11 months ago

@michael-ferreira2022, realizei o teste na branch master e não ocasionou o problema de hash, quais foram os steps executados?

  1. Org-1 inclui 3 documentos ao processo e, depois altera a ordem destes, modificando o documento posicionado na ordem 3 para a ordem 1;
  2. Org-1 envia o processo para o Org-2;
  3. Org-2 inclui mais dois documentos ao referido processo e, envia-o para o Org-1;
  4. Org-1 altera a ordem destes novos documentos incluídos pelo Org-2, modificando o documento posicionado na ordem 5 para a ordem 4 (ou seja, apenas houve uma troca do último para o penúltimo);
  5. Org-1 tenta enviar o processo para o Org-2 (obviamente, o sistema vai apresentar uma mensagem impedindo o envio, porque houve uma alteração nos documentos incluídos pelo Org-2);
  6. Org-1 reordena os documentos (modificando o documento posicionado na ordem 4 para a ordem 5, ou seja, da forma como estava anteriormente) e envia novamente o processo para o Org-2;
  7. Sistema não apresenta quaisquer mensagem de impedimento, reenviando novamente o processo para o Org-2.
michael-ferreira2022 commented 11 months ago

https://github.com/pengovbr/mod-sei-pen/assets/121502868/37ffc0dc-bdde-4b4d-abe2-0afdc4d38088

davidnascimento1618 commented 11 months ago

Prezado @michael-ferreira2022, ao retestar a issue, pude verificar que está ocasionando o seguinte erro: "Elemento 'hdnRelProtocoloProtocolo' não encontrado na página.", ao ingressar na tela de 'Ordenar Árvore do Processo', como você pode observar no .gif abaixo:

Elemento 'hdnRelProtocoloProtocolo' não encontrado na página

Ademais, corrigir o conflito apontado neste comentário.

michael-ferreira2022 commented 11 months ago

@davidnascimento1618 corrigido. Erro era devido o icone ter sido registro no arquivo InfraPaginaEsquema4 e durante merge o função que chamava o ícone mudou para o arquivo InfraPaginaEsquema3. Funcão foi adicionada ao arquivo e alerta parou de ser apresentado em tela.

davidnascimento1618 commented 10 months ago

Prezados, favor corrigir os seguintes pontos elencados nos respectivos pull requests:

mateussbh commented 7 months ago

@Mauro1706, Faz-se necessário atualizar a mensagem ao usuário (quando há o impedimento de trâmite por desordem de documentos). No último comentário que contém a mensagem é observado o texto do comentário: https://github.com/pengovbr/mod-sei-pen/issues/261#issuecomment-1632900283. Precisamos confirmar essa mensagem e atualizar o PR, e também confirmar se manterá o botão de reordenação automática dos documentos, conforme comentário https://github.com/pengovbr/mod-sei-pen/issues/261#issuecomment-1646242884. Este botão encontra-se no repositório do SEI.

psc @euviniciusmendonca

mateussbh commented 7 months ago

Conforme alinhamento na data de 01/02/24, foi definido que a mensagem final será a sugerida no comentário https://github.com/pengovbr/mod-sei-pen/issues/261#issuecomment-1632900283, conforme redação:

"Houve uma alteração na ordem dos documentos no processo, o que impede o reenvio de um processo que já foi tramitado pela plataforma. Portanto, é recomendado reordenar os documentos de acordo com a ordem original.

Caso você seja um usuário sem permissão para reordenar o processo, é necessário entrar em contato internamente para identificar quem possui essa permissão."

Para o botão de reordenar automaticamente, este deverá ser entregue em paralelo, e ficará aguardando a devida aprovação para sua implantação.

Mauro1706 commented 7 months ago

Alterado mensagem de validação para alteração de ordem.

https://github.com/pengovbr/mod-sei-pen/assets/83025710/7875eafd-4b52-4f2e-9852-7002d70c1355

sergioadao commented 6 months ago

Boa tarde, @Mauro1706 , @mateussbh , @carlosevmelo .......Issue validada com sucesso no ambiente SEI4/MYSQL. Implementação realizada conforme solicitado. Segue evidência:

https://github.com/pengovbr/mod-sei-pen/assets/87815724/1b4c5812-509e-4704-b497-32bb2ee793ab

Segue evidência SEI4/SQLSERVER:

https://github.com/pengovbr/mod-sei-pen/assets/87815724/797c6dee-9d89-46ce-923a-2d12640bc408

Segue evidência SEI4/ORACLE:

https://github.com/pengovbr/mod-sei-pen/assets/87815724/de35dbfc-b4b8-4553-92cf-788bb1e477bc

sergioadao commented 6 months ago

@Mauro1706 ......Boa tarde.....fiz um teste, a pedido do Mateus, com a seguinte sequência:

  1. Pegar um processo.
  2. Cria 2 documentos.
  3. Antes de tramitar, mudar a ordem do 1 pro 2. 4.Tramita o processo.
  4. Pegar o processo no org2 e devolve, sem fazer nada.
  5. No org1, adiciona 1 novo doc. Pega o de ordem 2 e coloca como último. E tenta tramitar novamente.

Ocorre o erro abaixo:

https://github.com/pengovbr/mod-sei-pen/assets/87815724/976ac77b-4e5e-42ee-bad9-84e72a03a9f7

NinetiesPaul commented 5 months ago

Recriado PR/branch do botão de reordenação (SEI) de https://github.com/pengovbr/sei/pull/301 para https://github.com/pengovbr/sei/pull/365 com correção de conflito

https://github.com/pengovbr/mod-sei-pen/assets/36829911/88251b09-c683-492c-9f4e-4f45cc8f0f58

sergioadao commented 5 months ago

Bom dia @NinetiesPaul e @mateussbh ...issue 261 PR 365 --- testada e confirmada o funcionamento correto. Teste realizada com SEI-OFICIAL/MYSQL .......

Seguem evidências:

https://github.com/pengovbr/mod-sei-pen/assets/87815724/f821390b-92a8-47ab-83e1-7f17b0b482c1

https://github.com/pengovbr/mod-sei-pen/assets/87815724/09f86468-34bc-41be-8351-cd2c5efa946d

https://github.com/pengovbr/mod-sei-pen/assets/87815724/9422a876-5fb8-4a5a-92de-ffef8601520c

yurioliveiramgi commented 3 months ago

Aguardando retorno do TRF4 sobre inclusão do botão de reordenação no core do SEI para realizar a validação negocial e entrega da referida issue.

Para ciência @mateussbh e @higodf