Issue #424

Closed
softwarepublico/softwarepublico#424
Created by Tallys Martins (Edited )

Validar com ministério os campos de instituição

Validar quais campos de instituição são obrigatórios no cadastro de uma nova instituição.

20 participants
  • 4f4b696adc8e8769f43b5c837b3ea23b?s=40&d=identicon
    Marisa Souza dos Santos @marisa

    @thiago vejam as observações da época: http://lappis.unb.br/redm/issues/834

    Choose File ...   File name...
    Cancel
  • 800e3af93350753db9dee1864cef10ff?s=40&d=identicon
    Thiago Ribeiro @thiago

    Reassigned to @pedrodelyra

    Choose File ...   File name...
    Cancel
  • 800e3af93350753db9dee1864cef10ff?s=40&d=identicon
    Thiago Ribeiro @thiago

    @pedrodelyra para lembrar de ver e comparar com os campos de instituição na modal.

    Choose File ...   File name...
    Cancel
  • Cae6de821da30189b2573c26aa4ddfa1?s=40&d=identicon
    Tallys Martins @tallysmartins

    @fabio1079 @lucianopc validar os campos com o que está descrito no redmine

    Choose File ...   File name...
    Cancel
  • 4f4b696adc8e8769f43b5c837b3ea23b?s=40&d=identicon
    Marisa Souza dos Santos @marisa (Edited )

    @thiago acabei de verificar que o campo cnpj realmente é obrigatório, não sei o pq nós o tiramos. É importante que vocês também verifiquem os resultados dos testes que realizei na época nos cadastros. Está na issue que eu anexei mas segue abaixo:

    TESTES REALIZADOS Inserir Instituição

    1 - Os campos não estão na ordem sugerida na especificação, sugerimos colocar o Tipo de Instituição logo como primeiro campo para carregar os campos de acordo com a escolha, acreditamos que a usabilidade fica melhor.

    2 - Ao invés de “Nome” colocar Razão Social

    3 - Sigla e Nome Fantasia são campos distintos.

    4 - Falta Sigla - Só mostrar campo “Sigla” para empresas tipo: pública. Campo Opcional

    5 - Nome fantasia será sempre opcional porém deve estar disponível no cadastro de qualquer empresa.

    6 - CNPJ não valida formato

    7 - CNPJ não valida tipo de caracteres (apenas números)

    8 - CNPJ não limita quantidade de caracteres (14 números)

    9 - Se País igual a “Brasil” popular Estado e Cidade de acordo com a base de cidades e estados do IBGE. Se país diferente de “Brasil” trazer campo descritivo para Estado e Cidade.

    10 - Só mostrar campo “SISP?” para empresas tipo: pública. Campo Obrigatório. Marcar sempre “Não” para os campos “SISP?”. Esse campo só pode ser alterado por Admins Admins Geral do sistema. Para todos os outros usuários do sistema o campo deve vir desabilitado.

    11 - Não tem o campo *Situação (Ativo/Inativo)

    12 - Não conseguimos incluir uma instituição pois dá erro no formato do CNPJ. Quando conseguimos um CNPJ válido fica dando mensagem de erro no captcha porém o mesmo encontra-se correto. E deu outro erro solicitando esfera, poder porém esses campos estavam preenchidos. Após esses problemas percebi que ele inseriu a instituição porém não mandou mensagem de inserção com sucesso.

    13 - Não conseguimos encontrar, de forma fácil e acessível a instituição após cadastro para a realização dos demais testes.

    14 - Tivemos que incluir a url na mão para conseguir acessar.

    15 - Permitiu cadastrar duas instituições com o mesmo CNPJ ( ver instituição teste e teste 123) Especificado: Antes de cadastrar/atualizar verificar a existência de empresas com o mesmo "CNPJ" e mesma "Razão Social" ou com Razão Social parecidas para evitar duplicidade de cadastro.

    16 - Deixou criar duas instituições com o mesmo nome e não sugeriu que ela pudesse estar cadastrando uma com um nome parecido com alguma cadastrada no sistema. (ver instituição teste e Teste). Especificado: Se a Razão Social for parecida, sugerir as empresas para que o usuário possa optar por escolher alguma parecida.

    17 - Qualquer usuário cadastrado pode alterar os dados de uma instituição tipo: privada.

    18 - Somente usuários de governo podem alterar os dados de uma instituição tipo: pública.

    Choose File ...   File name...
    Cancel
  • 4f4b696adc8e8769f43b5c837b3ea23b?s=40&d=identicon
    Marisa Souza dos Santos @marisa

    Ahhh @thiago acho que o tal Nome da Coorporação era Nome fantasia... Nome deveria ser razão social e o sigla permanece. É impotanque que vocês leiam este histórico da issue pra nos direcionarmos sobre as decisões tomadas na época. Decidimos hj pela manhã remover o nome da coorporação mas talvez seja apenas a nomenclatura que está errada. @peterlima tem como acompanhar isso?

    Choose File ...   File name...
    Cancel
  • 4f4b696adc8e8769f43b5c837b3ea23b?s=40&d=identicon
    Marisa Souza dos Santos @marisa

    @obsneto @nayanne @lavribeiro para conhecimento.

    Choose File ...   File name...
    Cancel
  • 800e3af93350753db9dee1864cef10ff?s=40&d=identicon
    Thiago Ribeiro @thiago

    @lucianopc e @pedrodelyra vou encaminhar para vocês corrigirem isso na sexta feira ok?

    Choose File ...   File name...
    Cancel
  • Cae6de821da30189b2573c26aa4ddfa1?s=40&d=identicon
    Tallys Martins @tallysmartins

    Reassigned to @lucianopc

    Choose File ...   File name...
    Cancel
  • Cae6de821da30189b2573c26aa4ddfa1?s=40&d=identicon
    Tallys Martins @tallysmartins

    mentioned in issue #443

    Choose File ...   File name...
    Cancel
  • Cae6de821da30189b2573c26aa4ddfa1?s=40&d=identicon
    Tallys Martins @tallysmartins

    Status changed to closed

    Choose File ...   File name...
    Cancel
  • 4f4b696adc8e8769f43b5c837b3ea23b?s=40&d=identicon
    Marisa Souza dos Santos @marisa

    Tentei cadastrar uma instituição no dev e aparece a seguinte tela Era uma instituição como nome "Agência Nacional do Petróleo, Gás Natural e Biocombustíveis" visando testar o comportamento no caso de Razão social com o mesmo nome de uma instituição já cadastrada. Eu sugiro fazermos uma reunião presencial para fecharmos a validação deste cadastro.

    Choose File ...   File name...
    Cancel
  • 4f4b696adc8e8769f43b5c837b3ea23b?s=40&d=identicon
    Marisa Souza dos Santos @marisa

    Status changed to reopened

    Choose File ...   File name...
    Cancel
  • 4f4b696adc8e8769f43b5c837b3ea23b?s=40&d=identicon
    Marisa Souza dos Santos @marisa

    Milestone removed

    Choose File ...   File name...
    Cancel
  • D06c8bf1c58ad143edc3f9314c51ed9f?s=40&d=identicon
    Luciano Prestes @lucianopc
    Choose File ...   File name...
    Cancel
  • 800e3af93350753db9dee1864cef10ff?s=40&d=identicon
    Thiago Ribeiro @thiago

    @lmatos pode homologar?

    Choose File ...   File name...
    Cancel
  • 71c7041f7667454f3d6bf8f889e2d598?s=40&d=identicon
    Luiz Matos @lmatos

    @thiago Posso sim, vou homologar tudo quando o dev voltar

    Choose File ...   File name...
    Cancel
  • B4c7afc2cdffea95a8c2d0387acedc6b?s=40&d=identicon
    Paulo Meireles @paulormm

    Reassigned to @lmatos

    Choose File ...   File name...
    Cancel
  • B4c7afc2cdffea95a8c2d0387acedc6b?s=40&d=identicon
    Paulo Meireles @paulormm

    Milestone changed to Prioridades para o Encerramento

    Choose File ...   File name...
    Cancel
  • B4c7afc2cdffea95a8c2d0387acedc6b?s=40&d=identicon
    Paulo Meireles @paulormm

    Todas que estiverem fechadas no Milestone de Encerramento serão homologadas pelo MP. Vou fechar para nosso controle interno de encerramento do projeto.

    @kanashiro , @marcosronaldo e @melissawen : para conhecimento.

    Choose File ...   File name...
    Cancel
  • B4c7afc2cdffea95a8c2d0387acedc6b?s=40&d=identicon
    Paulo Meireles @paulormm

    Status changed to closed

    Choose File ...   File name...
    Cancel
  • 4f4b696adc8e8769f43b5c837b3ea23b?s=40&d=identicon
    Marisa Souza dos Santos @marisa (Edited )

    Na validação do cadastro de instituição encontramos os seguintes problemas:

    • Sigla só deve existir para instituição pública,
    • Não há limitadores de tamanho nos campos: razao social, nome fantasia e cidade podendo gerar estouro do tamanho do campo
    • SISP e SIORG só devem ser incluído/editado por admins do ambiente
    • Sigla está muito escondido no designer da tela
    • Não sugere nomes parecidos de razão social para evitar cadastro duplicado
    Choose File ...   File name...
    Cancel
  • 4f4b696adc8e8769f43b5c837b3ea23b?s=40&d=identicon
    Marisa Souza dos Santos @marisa

    Status changed to reopened

    Choose File ...   File name...
    Cancel
  • Cae6de821da30189b2573c26aa4ddfa1?s=40&d=identicon
    Tallys Martins @tallysmartins

    A evolução completa do cadastro de instituições deixou de ser uma prioridade quando removemos o SISP do escopo. Mesmo assim, observe que dedicamos um esforço para aplicar melhorias na funcionalidade. A tela de cadastro/edição, por exemplo, está bem mais agradável aos usuários. Quanto aos pontos citados: 1 - SIGLA será removida para instituições privadas: será feita uma correção no javascript 2 - Limitadores dos campos não são necessários. Não entendemos a questão quanto ao tamanho do campo no banco de dados, pois o framework já trata isso. 3 - Quanto ao design da tela poderá ver com o @peterlima 4 - Somente admins tem acesso ao cadastro/edição de uma nova instituição. Testei com um usuário comum e está OK. 5 - Quanto à validação de razão social e CNPJ, será vista esta questão para validar com âmbos os campos.

    Todos os pontos poderão ser vistos com a equipe na quinta-feira.

    Choose File ...   File name...
    Cancel
  • 4f4b696adc8e8769f43b5c837b3ea23b?s=40&d=identicon
    Marisa Souza dos Santos @marisa

    Amanhã conversamos sobre isso... um cmapo semlimitdor pode gerar estouro no banco, aconteceu aqui, simplesmente enchi de lixo "55dfsaaaaaaaaaaaasddddddddddddddddddsdfregha" e deu erro :-) Amanhã podemos ver isso juntos.

    Choose File ...   File name...
    Cancel
  • B4c7afc2cdffea95a8c2d0387acedc6b?s=40&d=identicon
    Paulo Meireles @paulormm

    Apenas registrando que o cadastro de instituições está em produção desde janeiro: https://softwarepublico.gov.br/social/plugin/gov_user/create_institution_admin. Ele foi colocando junto, quando subimos o protótipo do catálogo do SISP: https://novo.sisp.gov.br/.

    Choose File ...   File name...
    Cancel
  • 46c36a7ea48b9a42086ce42e15f40cba?s=40&d=identicon
    Peter Lauro de Lima @peterlima (Edited )

    @alvarofernandoms @dylanguedes

    Uma das demandas, será reposicionar o campo "sigla" para ficar ao lado de "Razão Social".

    Já o campo "Nome Fantasia", ficará abaixo dos campos citados acima.

    Segue print, detalhando como deve aparecer.

    Lembrando que:

    • O ajuste será no admin e na modal.
    • Verificar a largura sugerida (print anexo) dos novos campos

    @melissawen a para acompanhamento.

    Choose File ...   File name...
    Cancel
  • 0857617ca13d090825c8eb4228efb6ae?s=40&d=identicon
    Marcos Pereira @marcosronaldo

    Reassigned to @alvarofernandoms

    Choose File ...   File name...
    Cancel
  • 0857617ca13d090825c8eb4228efb6ae?s=40&d=identicon
    Marcos Pereira @marcosronaldo

    mentioned in issue #145

    Choose File ...   File name...
    Cancel
  • 79ebee40dd44c661fc4ad27b3656be68?s=40&d=identicon
    Álvaro Fernando Matos de Souza @alvarofernandoms

    mentioned in merge request !257

    Choose File ...   File name...
    Cancel
  • B4c7afc2cdffea95a8c2d0387acedc6b?s=40&d=identicon
    Paulo Meireles @paulormm

    @melissawen : MR !257 ;)

    Choose File ...   File name...
    Cancel
  • 79ebee40dd44c661fc4ad27b3656be68?s=40&d=identicon
    Álvaro Fernando Matos de Souza @alvarofernandoms

    mentioned in merge request !260

    Choose File ...   File name...
    Cancel
  • 0743c2eb05e68c92baa5f8c498995a20?s=40&d=identicon
    Daniela Feitosa @daniela
    Choose File ...   File name...
    Cancel
  • 0857617ca13d090825c8eb4228efb6ae?s=40&d=identicon
    Marcos Pereira @marcosronaldo

    Homologar em dev.

    Choose File ...   File name...
    Cancel
  • 0857617ca13d090825c8eb4228efb6ae?s=40&d=identicon
    Marcos Pereira @marcosronaldo

    Status changed to closed

    Choose File ...   File name...
    Cancel
  • 4f4b696adc8e8769f43b5c837b3ea23b?s=40&d=identicon
    Marisa Souza dos Santos @marisa

    @ioneoliveira vamos validar juntas.

    Choose File ...   File name...
    Cancel
  • 4f4b696adc8e8769f43b5c837b3ea23b?s=40&d=identicon
    Marisa Souza dos Santos @marisa (Edited )

    Pessoal, Após as alterações que fizeram conseguimos validar todo o fluxo de uma intituição. As pendências que encontramos no cadastro são:

    • Não conseguimos editar o cadastro de ums intituição, você clica no botão de salvar e nada acontece.
    • Na exclusão de uma instituição dá a mensagem de Erro interno do Servidor, porém parece que a exclusão aconteceu normalmente
    • O código do SIORG aparece com o * de obrigatório na edição, na inclusão não aparece. Eu consegui incluir um órgão sem o SIORG deu certo, o comportamento está ok. Na verdade acho que nesse caso é só remover o * da tela de edição pq é um campo opcional mesmo.
    Choose File ...   File name...
    Cancel
  • 4f4b696adc8e8769f43b5c837b3ea23b?s=40&d=identicon
    Marisa Souza dos Santos @marisa (Edited )

    Foto da edição

    Choose File ...   File name...
    Cancel
  • 0857617ca13d090825c8eb4228efb6ae?s=40&d=identicon
    Marcos Pereira @marcosronaldo

    Status changed to reopened

    Choose File ...   File name...
    Cancel
  • B4c7afc2cdffea95a8c2d0387acedc6b?s=40&d=identicon
    Paulo Meireles @paulormm

    Reassigned to @marcosronaldo

    Choose File ...   File name...
    Cancel
  • B4c7afc2cdffea95a8c2d0387acedc6b?s=40&d=identicon
    Paulo Meireles @paulormm

    @marcosronaldo : por favor, dar um parece quanto a isso. Veja se o @tallysmartins e o @gabrielssilva ajudam, se precisar ;)

    Choose File ...   File name...
    Cancel
  • 0857617ca13d090825c8eb4228efb6ae?s=40&d=identicon
    Marcos Pereira @marcosronaldo

    Vamos tentar reproduzir o problema e trabalhar nele agora a tarde.

    Choose File ...   File name...
    Cancel
  • 0857617ca13d090825c8eb4228efb6ae?s=40&d=identicon
    Marcos Pereira @marcosronaldo

    @marisa, não conseguimos reproduzir o problema de exclusão de uma instituição. O problema do botão será resolvido na próxima atualização, assim como a remoção do (*) no campo de código SIORG.

    Choose File ...   File name...
    Cancel
  • 4f4b696adc8e8769f43b5c837b3ea23b?s=40&d=identicon
    Marisa Souza dos Santos @marisa

    Blz heheh aqui aconteceu em produção, cadastramos duas empresas privadas e nas duas, no momento da exclusão deu isso, mas excluí pela tela de administração https://softwarepublico.gov.br/social/admin/organizations

    Choose File ...   File name...
    Cancel
  • Def69c998857099b7bc246389e6ad936?s=40&d=identicon
    Lucas Kanashiro @kanashiro

    Status changed to closed

    Choose File ...   File name...
    Cancel
  • B4c7afc2cdffea95a8c2d0387acedc6b?s=40&d=identicon
    Paulo Meireles @paulormm

    @marisa e @lmatos : por favor, homologar em produção

    • OBS: o ambiente Dev não foi atualiazdo completamente, pois precisará ter os mesmos recursos de prod para servir de homologação, no futuro para vocês).
    Choose File ...   File name...
    Cancel
  • 4f4b696adc8e8769f43b5c837b3ea23b?s=40&d=identicon
    Marisa Souza dos Santos @marisa

    O único problema que continua é o seguinte:

    • Na exclusão de uma instituição dá a mensagem de Erro interno do Servidor, porém parece que a exclusão aconteceu normalmente

    O restante está ok.

    Choose File ...   File name...
    Cancel
  • B4c7afc2cdffea95a8c2d0387acedc6b?s=40&d=identicon
    Paulo Meireles @paulormm

    mentioned in issue #875

    Choose File ...   File name...
    Cancel
  • B4c7afc2cdffea95a8c2d0387acedc6b?s=40&d=identicon
    Paulo Meireles @paulormm (Edited )

    @marisa e @lmatos : no escopo desta issue, está resolvido/homologado, ao validarem os campos.

    Como vocês acharam um problema, assim, eu criei uma nova issue #875 , no backlog, mas estará em nosso radar para tentarmos via comunidade ou estagiários da UnB.

    Choose File ...   File name...
    Cancel
  • 4f4b696adc8e8769f43b5c837b3ea23b?s=40&d=identicon
    Marisa Souza dos Santos @marisa

    homologado e aberta a issue #875 para cuidar da mensagem de erro na exclusão.

    Choose File ...   File name...
    Cancel
  • 4f4b696adc8e8769f43b5c837b3ea23b?s=40&d=identicon
    Marisa Souza dos Santos @marisa

    Comforme combinamos em reunião abrimos a issue #879 para tratar palavra em inglês na tela de busca de instituições

    Choose File ...   File name...
    Cancel