pengovbr / mod-sei-pen

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

Limitação de Repositórios/Unidades Para o Envio de Processo #196

Open LeandroMedeirosLima opened 1 year ago

LeandroMedeirosLima commented 1 year ago

Sua solicitação de melhoria está relacionada a um problema? Por favor descreva. Atualmente qualquer unidade pode enviar processos externos para repositório/unidades aptas para o recebimento de processos e documentos avulsos.

Um gestor de negócio reportou que isso pode ser um problema dependendo do usuário e pode levar erro no envio do processo.


Descreva a solução que você gostaria A possibilidade do gestor de negócio restringir quais repositórios de estrutura uma unidade pode escolher como destino de envio de processos.


Contexto adicional Reunião com os gestores do TJMG que propuseram o requisito: https://mtegovbr-my.sharepoint.com/:v:/g/personal/vinicius_mendonca_economia_gov_br/EQJJpJ3y9bRJo6fJQ0JOANoBFg-X8gAPCGkfocInMXHHfQ

caduvieira commented 1 year ago

O arquivo não está liberado para leitura.

euviniciusmendonca commented 1 year ago

https://mtegovbr-my.sharepoint.com/:v:/g/personal/vinicius_mendonca_economia_gov_br/EQJJpJ3y9bRJo6fJQ0JOANoB9BhUL13IRCNwGiKTJy2YSQ?e=ACyHil

@caduvieira, co?nfirma o acesso

euviniciusmendonca commented 1 year ago

Solicita-se um vídeo com a evidência da melhoria.

caduvieira commented 1 year ago

@Mauro1706 por favor pode gerar esse vídeo?

Mauro1706 commented 1 year ago

Segue evidências de funcionamento.

https://github.com/supergovbr/mod-sei-pen/assets/83025710/9f579fb2-2e65-48b6-81a7-81b095e8b7c7

https://github.com/supergovbr/mod-sei-pen/assets/83025710/f17b14a7-50d4-4c36-b9a9-d4edfd6bb988

davidnascimento1618 commented 1 year ago

Prezado, @Mauro1706, vi que os itens defeituosos encontrados na issue em questão, que foram descritos e evidenciados no comentário deste pull request, ainda continuam. Houve atualização no PR #274?

Mauro1706 commented 1 year ago

@davidnascimento1618 a branch ficou com conflito e não subiu as correções pertinentes

davidnascimento1618 commented 1 year ago

Prezado @Mauro1706, após novos testes realizados, foram identificados três problemas, que estão detalhados e evidenciados neste comentário aqui.

MatheusMarques80 commented 1 year ago

Não foi possível concluir os novos ajustes solicitados pelo @davidnascimento1618 dentro do time box da sprint atual. Foi feito um estudo da funcionalidade como um todo e para a correção dos pontos mencionados será necessário mais alguns dias.

MatheusMarques80 commented 1 year ago

Foi corrigido os seguintes itens:

  1. Ao adicionar unidades restritivas a algum repositório (a partir do segundo adicionado em diante), o sistema estava deletando todas as unidades restritivas da lista.

  2. Ao editar um mapeamento existente para excluir uma unidade restritiva da lista, o sistema estava removendo os repositórios não correlacionados, ou seja, considerando a seguinte relação: A -> X; B -> Y; e C -> Z, caso o usuário excluísse a unidade X, ao salvar e voltar à tela (via edição), o sistema estava removendo os repositórios B e C, em vez de remover apenas o A.

  3. Tempo que um repositório e uma unidade estão sendo carregados

  4. Exibição do ícone de carregamento

https://github.com/pengovbr/mod-sei-pen/assets/13146046/a73f2b1a-250f-417c-bcc4-0fcf04a1898a

davidnascimento1618 commented 1 year ago

@MatheusMarques80, favor corrigir o problema apontado neste comentário do pull request #361,

gledson-sousa commented 11 months ago

Realizado a correção no repositório de estruturas quando for um valor nulo "Repositório de Estruturas não Encontrado."

https://github.com/pengovbr/mod-sei-pen/assets/121505958/408fccc3-3dce-45e9-a740-76568c6dfe3a

davidnascimento1618 commented 10 months ago

Prezados, favor corrigir os conflitos mencionados no pull request em questão.

Mauro1706 commented 8 months ago

Segue evidência da funcionalidade:

https://github.com/pengovbr/mod-sei-pen/assets/83025710/cc4c69db-1cbb-4120-bad7-1d8982fe662f

mateussbh commented 8 months ago

Prezados, favor incluir o teste funcional da issue.

Mauro1706 commented 7 months ago

https://github.com/pengovbr/mod-sei-pen/assets/83025710/8a96c245-b4df-4731-bc2d-4868a09d0ae5

sergioadao commented 6 months ago

@Mauro1706 ....Bom dia ...erro encontrado quando uma unidade é incluída na lista de restrição.....

  1. Permite unidade duplicada
  2. Não permite remoção.

Segue evidência:

https://github.com/pengovbr/mod-sei-pen/assets/87815724/ddbb2157-f6e3-427e-812d-fc29f280b5d6

Mauro1706 commented 6 months ago

Resolvido duplicidade de cadastro.

https://github.com/pengovbr/mod-sei-pen/assets/83025710/6265176f-44b1-4c83-9974-dc277e7b14a0

sergioadao commented 6 months ago

Bom dia @Mauro1706 e @mateussbh ...testado e confirmado o funcionamento correto. Segue evidência:

OBS: Trocar o nome do menu de Protocolo Eletrônico Nacional para Tramita GOV.BR

https://github.com/pengovbr/mod-sei-pen/assets/87815724/843791fc-6bba-4db6-8914-73c762b492ed

amelquia commented 4 months ago

06/05/24 Necessário realizar uma limpeza de arquivos que já foram ajustados nessa issue e, deixar apenas os arquivos pertinentes à mesma.

amelquia commented 4 months ago

12/05/24 Issue não finalizada e migrada para a Sprint 42.

amelquia commented 3 months ago

27/05/24 Sprint Planning - Sprint 43 realizada. Issue transferida para a Sprint 43 para dar sequencia no fluxo dos status. Status atual: "Validação Técnica." Nova estimativa: 2 dias.

Mauro1706 commented 3 months ago

Segue evidencia da funcionalidade: Segue Branch: https://github.com/pengovbr/mod-sei-pen/pull/510

https://github.com/pengovbr/mod-sei-pen/assets/83025710/67c65207-6ef2-42e1-8d11-931bf81c597f

amelquia commented 3 months ago

03/06/24 No dia 31/05, o Mauro refez os testes e colocou o vídeo para que o Sérgio possa entender que não há erro. A issue está disponível para validação técnica.

Mauro1706 commented 3 months ago

Refeito o PR: https://github.com/pengovbr/mod-sei-pen/pull/511

amelquia commented 3 months ago

10/06/24 Card movido para a Sprint 44 e movido para o status "Em andamento" pois foram identificados alguns ajustes para serem realizados.

amelquia commented 3 months ago

11/06/24 Resolvida a questão do java script que solucionava a estrutura e unidade. Outros erros foram gerados devido ao componente ser antigo. Resolvendo a parte do tramitar das unidades e resolução de bugs.

Mauro1706 commented 3 months ago

Resolvido a seleção de repositórios que limpa a seleção de unidades. PR: https://github.com/pengovbr/mod-sei-pen/pull/518

yurioliveiramgi commented 3 months ago

Foi identificado que a limitação de repositórios funciona corretamente por meio do campo em que o nome da unidade será inserido. Entretanto, ao clicar no ícone "Consultar Organograma" este não realiza o carregamento.

Image

@mateussbh , @Mauro1706 e @euviniciusmendonca , PSC

amelquia commented 2 months ago

21/06/24 Movimento da issue em Em Análise P.O. para a Sprint 45 .

amelquia commented 2 months ago

24/06/24 Após explicação do @yurioliveiramgi , o card foi movido para Ajustes Necessários.

Mauro1706 commented 2 months ago

Organograma resolvido no PR: https://github.com/pengovbr/mod-sei-pen/pull/528

Mauro1706 commented 2 months ago

Resolvido restrição de envio para bloco e organograma! https://github.com/pengovbr/mod-sei-pen/pull/537

amelquia commented 2 months ago

08/07/24 Planning realizada. Card movido para a Sprint46.

sergioadao commented 2 months ago

Bom dia @Mauro1706 e @amelquia .... Segue erro encontrado no mapeamento de unidades após o pull do PR 537.

Segue evidência:

https://github.com/pengovbr/mod-sei-pen/assets/87815724/1de15700-c56a-41da-a443-38f138478c15

Mauro1706 commented 2 months ago

Realizado a correção no PR https://github.com/pengovbr/mod-sei-pen/pull/539 com a separação dos métodos de autocompletar no cadastro de unidades e envio de processo!

caduvieira commented 2 months ago

@sergioadao testar novo PR

sergioadao commented 2 months ago

Bom dia @Mauro1706 e @amelquia ..... segue testes realizados com as versões 4.0.12 , 4.1.1 e 4.1.3.

No caso, da versão 4.1.3 , surgiu um novo erro na criação de um processo. Segue evidência.

https://github.com/pengovbr/mod-sei-pen/assets/87815724/ed44c69b-25b4-488a-a488-8c47f11c17b4

https://github.com/pengovbr/mod-sei-pen/assets/87815724/5dca54e0-99f0-4754-bacb-025a67f0737e

https://github.com/pengovbr/mod-sei-pen/assets/87815724/ce46f6fd-4c25-419f-9457-6e0cb40fb1e6

euviniciusmendonca commented 2 months ago

@amelquia @yurioliveiramgi @Mauro1706

Conforme explicado em reunião, a funcionalidade organograma será ocultada, visto que não é utilizada pela maioria dos usuários. Dessa forma, eu acredito que o impeditivo para conclusão dessa issue foi removido.

OBS: Posteriormente, vamos analisar a viabilidade de disponibilizar novamente a funcionalidade.

Atenciosamente,

euviniciusmendonca commented 2 months ago

@yurioliveiramgi

Realizar o mapeamento do impacto dessa alteração na documentação do Tramita GOV.BR.

yurioliveiramgi commented 2 months ago

A funcionalidade prevista para a issue está correta. No dia de hoje foi liberado um patch que resolveu o problema do organograma para as versões 4.1.x. Diante disso, caso seja mantida a decisão de ocultar o organograma (issue #540), a issue está validada negocialmente. Caso contrário, será necessário aguardar o merge do PR para posterior validação.

@euviniciusmendonca PSC e validação.

yurioliveiramgi commented 2 months ago

16/07/24 - Issue validada, funcionamento conforme esperado. Movida para Aceito Negocial.

@Mauro1706 , @amelquia , @euviniciusmendonca e @mateussbh , PSC

yurioliveiramgi commented 1 month ago

A presente issue está sendo encaminhada para realização de ajustes. Na realização de alguns testes, foi verificada a necessidade de algumas correções:

  1. Ao realizar a limitação de apenas estruturas (mantendo o campo de unidades vazio), Figura 01, ao tentar realizar o envio de um trâmite, a limitação de repositórios de estruturas é respeitada, Figura 02. Entretanto, nenhuma unidade é exibida como destino, Figura 03.

    • [x] Caso o gestor preencha apenas o campo "Restringir às Estruturas" deve ser compreendido que todas as unidades pertencentes ao(s) repositório(s) selecionados deve(m) ser exibido(s). Ou seja, existirá apenas uma limitação de repositórios de estruturas e não de unidades; Figura 01 image Figura 02 image Figura 03 image
  2. Correção gramatical no texto exibido nas telas "Novo Mapeamento de Unidades" (Figura 04) e "Editar Mapeamento de Unidades" (Figura 05).

  1. Aproveito para solicitar a alteração do título da tela "Mapeamento de Unidade" para "Mapeamento de Unidades" image

    @euviniciusmendonca , @amelquia , @mateussbh , @Mauro1706 , PSC.

amelquia commented 1 month ago

@yurioliveiramgi Como essa issue foi validada e dada como Aceito Negocial, peço a gentileza de abrir uma nova issue para análise e correções.

yurioliveiramgi commented 1 month ago

@euviniciusmendonca, PSC

euviniciusmendonca commented 1 month ago

Vamos alinhar isso na Daily.

On Tue, Jul 30, 2024 at 9:05 AM Yuri Oliveira @.***> wrote:

@euviniciusmendonca https://github.com/euviniciusmendonca, PSC

— Reply to this email directly, view it on GitHub https://github.com/pengovbr/mod-sei-pen/issues/196#issuecomment-2258189002, or unsubscribe https://github.com/notifications/unsubscribe-auth/AOOFLIPZVYSVC4JTINPFZLDZO56SBAVCNFSM6AAAAAASRC5446VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENJYGE4DSMBQGI . You are receiving this because you were mentioned.Message ID: @.***>

amelquia commented 1 month ago

30/07/24 - Daily Conforme solicitado pelo @euviniciusmendonca, a fim de ganharmos celeridade com as tratativas para a liberação da release 3.7.0, a issue será alterada para "Ajustes Necessários" e, será tratada na Sprint atual - Sprint 47.

Mauro1706 commented 1 month ago

Realizado as correções apontadas pelo @yurioliveiramgi no PR https://github.com/pengovbr/mod-sei-pen/pull/565 Segue evidência da correção:

https://github.com/user-attachments/assets/a6fc7bf3-5fed-4d6b-ba83-d67565063e23

yurioliveiramgi commented 1 month ago

Issue validada negocialmente. Funcionamento conforme o previsto.

@amelquia , @mateussbh e @euviniciusmendonca