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

261 melhoria da mensagem alteração na ordem de documentos do processo indicar ordem #269

Conversation

michael-ferreira2022
Copy link
Contributor

@michael-ferreira2022 michael-ferreira2022 commented Apr 24, 2023

@guilhermeadc
Copy link
Contributor

Pull Request atualizado na branch review/sprint7-16 para validação em ambiente de homologação.

@guilhermeadc
Copy link
Contributor

Esta Issue foi marcada com contendo Impedimentos devido a problemas de formatação da mensagem indicativa de mudança de ordem dos documentos. Esta é uma limitação do componente de Barra de Progresso utilizada pelo próprio SEI e, com isso, o desenvolvimento solicitado ficou limitado.

Será necessário uma nova avaliação pela equipe da CGSIS para passar uma outra solução técnica para este problema de formatação.

@davidnascimento1618
Copy link
Contributor

Foi verificado que a mensagem que está sendo exibida para o usuário, está ficando muito extensa, principalmente quando há uma quantidade considerável de documentos anexados ao processo, esta mensagem, além de apontar o problema, tem o intuito de orientar o usuário para a devida solução deste, isto é, da ordenação íntegra destes documentos dentro do processo. Analisaremos internamente uma melhor forma de tratar tal questão.

@davidnascimento1618
Copy link
Contributor

Foi acertado que a mensagem será exibida em partes para o usuário, ou seja, inicialmente, somente a primeira parte da mensagem será exibida, independentemente da quantidade de movimentações que tiveram sido realizadas na ordem dos documentos e, portanto, o usuário irá realizar a devida correção conforme orientação, em seguida, será exibida a próxima e, assim sucessivamente, até que todo o conjunto tenha sido reorganizado, conforme a ordem original do processo na qual foi recebido.
Abaixo segue um exemplo da mensagem cujo o processo possui 6 documentos e foi alterado a ordem do documento de posição 6 para a posição 1.

Comportamento atual:

Inconsistência identificada no documento de ordem '3' do processo tramitado por este NRE, '0000027629112023', com protocolo '99990.000002/2023-55': Não foi possível enviar o processo '99990.000002/2023-55' 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 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 3. A ordem do documento 000004 foi modificada na árvore do processo, mudando da posição 3 para a posição 4. A ordem do documento 000005 foi modificada na árvore do processo, mudando da posição 4 para a posição 5. A ordem do documento 000006 foi modificada na árvore do processo, mudando da posição 5 para a posição 6. A ordem do documento 000007 foi modificada na árvore do processo, mudando da posição 6 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.

Como deveria ser:

Inconsistência identificada no documento de ordem '6' do processo tramitado por este NRE, '0000027629112023', com protocolo '99990.000002/2023-55': Não foi possível enviar o processo '99990.000002/2023-55' 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 000007 foi modificada na árvore do processo, mudando da posição 6 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.

@pcborges pcborges requested review from davidnascimento1618 and removed request for pcborges June 21, 2023 15:07
Copy link
Contributor

@davidnascimento1618 davidnascimento1618 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Caro @michael-ferreira2022, foi verificado que a mensagem de alteração da ordem nos documentos, não está orientando o usuário corretamente, no exemplo da evidência abaixo, mostra um processo com três documentos onde um documento foi deslocado da posição 3 para a posição 1 e, consequentemente, o documento que estava na posição 1 foi para a posição 2 e o de posição 2 para a posição 3, no entanto, o que aconteceu é que o sistema informou que o documento que estava na posição 3 mudou da posição 1 para a 2, o que está errado, ademais, mesmo realizando a alteração sugerida pela mensagem e reenviando o processo, o sistema exibiu a próxima mensagem sem quaisquer sugestões de resolução.

Mensagem.de.alteracao.na.ordem.de.documentos.nao.esta.orientando.o.usuario.corretamente.mp4

Copy link
Contributor

@davidnascimento1618 davidnascimento1618 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Prezado @michael-ferreira2022, no reteste foi verificado que, a mensagem de alteração da ordem nos documentos, continua incorreta, no exemplo da evidência abaixo, mostra um processo com três documentos onde um documento foi deslocado da posição 3 para a posição 1 e, consequentemente, o documento que estava na posição 1 foi para a posição 2 e o de posição 2 para a posição 3, no entanto, o que aconteceu é que o sistema informou que o documento que estava na posição 3 mudou da posição 1 para a 2, o que está errado, ademais, o sistema acusou um número de documento que não existe na lista em questão, ao invés dele informar que seria o documento de '0000012' que foi alterado, ele indicou que seria o documento de número '000014'.

Mensagem.de.alteracao.na.ordem.de.documentos.nao.esta.orientando.o.usuario.corretamente_1.mp4

Copy link
Contributor

@davidnascimento1618 davidnascimento1618 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@michael-ferreira2022:

  1. Adicionar um caso de teste, conforme o cenário descrito nos steps do comentário da issue em questão;
  2. Ajustar o teste conforme o padrão, no tocante à nomenclatura dos arquivos e classes, ou seja, o certo seria 'TramiteReordenarProcessoTest' ao invés de 'ReordenarProcesso';
  3. Corrigir a seguinte falha na execução do teste em questão:

image

  1. Após a execução bem sucedida dos testes, gravar um evidência em vídeo e anexar na issue correlacionada.

Copy link
Contributor

@davidnascimento1618 davidnascimento1618 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Prezado, @michael-ferreira2022, ao tentar atualizar a branch em meu ambiente local, houve alguns conflitos nos arquivos: ProcessoEletronicoRN.php e ExpedirProcedimentoRN.php, como você pode observar no print abaixo, assim que corrigi-los, favor sinalizar para que eu possa realizar os retestes descritos neste no comentário supracitado.

image

Copy link
Contributor

@davidnascimento1618 davidnascimento1618 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Prezados, conforme exibido no print abaixo, ao tentar atualizar a branch localmente, houve alguns conflitos nas classes: ProcessoEletronicoRN e ExpedirProcedimentoRN, assim que corrigi-los, favor sinalizar para que eu possa realizar os retestes na issue correlacionada.

image

Copy link
Contributor

@davidnascimento1618 davidnascimento1618 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Prezados, favor verificar os seguintes itens e corrigi-los:

  • 1. Atribuir um nome mais intuitivo ao teste (isso inclui nomenclatura dos arquivos e classes correlacionadas), ou seja, uma sugestão seria: 'TramiteReordenarDocumentosProcessoTest' ao invés de 'ReordenarProcesso';
  • 2. Adicionar um test case ao functional test, conforme o cenário descrito nos steps deste comentário;
  • 3. O '$this->assertTrue(true)' (linhas 47 e 113) não está tendo nenhuma ação no teste, deveria validar a ordem dos documentos ao final da reordenação e, consequentemente, o envio do processo para o org-2 sem quaisquer impedimentos, ou seja, sem qualquer mensagem de erro/impedimento pertinente à inconsistência na ordem dos documentos;
  • 4. Não foi encontrado o teste funcional para a versão 3 do SEI;
  • 5. Não foi encontrado o teste funcional para a versão 4 do SEI.

@Mauro1706
Copy link
Contributor

Resolvido no PR #425

@Mauro1706 Mauro1706 closed this Jan 26, 2024
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.

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