Issue #298

Closed
softwarepublico/softwarepublico#298
Created by Marisa Souza dos Santos (Edited )

Permitir apenas empresas cadastradas no campo de órgão do relato

Não Permitir texto simples no campo. Apenas empresas cadastradas no campo de órgão do relato poderão ser exibidas.

Arrumar formatação dos campos numéricos.

13 participants
  • Cae6de821da30189b2573c26aa4ddfa1?s=40&d=identicon
    Tallys Martins @tallysmartins

    Reassigned to @tallysmartins

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

    @tallysmartins ficou combinado que naquele campo de selecionar a empresa/instituição nós colocariamos um select_box com filtro.

    além disso formatar os campos numéricos.

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

    o @fabio1079 disse que fica sem salvar o nome da empresa/instituição se ele não selecionar uma das que são auto-completadas, logo não iria lixo!

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

    para poder ajudar vamos remover o campo texto e deixar um select_box com filtro.

    logo a funcionalidade está ok, apenas vamos fazer essa melhoria.

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

    @marisa

    Marisa, vimos que para Permitir apenas empresas cadastradas no campo de órgão do relato o campo deverá ser obrigatório.

    Ex: quando o usuário escreve uma instituição inválida aparece a mensagem embaixo que a instituição está inválida e por mais que ele envie essa infomação inválida a informação na avaliação fica em branco ou seja não salva o "lixo" digitado que for inválido.

    Isso porque hoje esse campo não é obrigatório!

    Mas tem o porém de se tornar obrigatório, se a empresa/instituição dele não existir ele vai escolher qualquer uma e isso pode trazer informação inválida. Por isso pensamos que deixar em branco por enquanto é a melhor forma.

    @peterlima e @melissawen poderia dar a opinião de vocês? vamos discutir isso juntos.

    No problema em questão da issue que é permitir selecionar apenas empresas cadastradas, podemos deixar o campo como obrigatório ou deixar como está.

    Creio que quando o usuário puder criar a sua instituição/empresa (feature nova discutida com o peter e com vcs) esse problema será resolvido.

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

    @thiago Na minha opinião o campo deve ser opcional porém só permitir empresas cadastradas. E se não tiver a empresa do usuário que está relatando? Nesse caso já discutimos na última reunião que deveria ter uma opçaõ para ele cadastrar a empresa. Não tem como deixar obrigatório e ele escolher "qualquer uma", temos que dar opção para ele cadastrar.

    Choose File ...   File name...
    Cancel
  • C8b72d0556872a2aea21e8fed0a72001?s=40&d=identicon
    Melissa Wen @melissawen

    @thiago não tem como no backend fazer assim: se nome válido, salva o órgão. Se nome inválido, deixa em branco e salva em branco?

    @thiago não esquece de mapear isso como evolução da funcionalidade ;).

    ps.: Vamos juntar essa discussão com a da navegação (botão voltar) junto com o @peterlima? aproveita ele presencial aí, agenda um horário e me diz! Estou -2h do Brasil agora com o horário de verão ;)

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

    @marisa então acho que já podemos iniciar a conversa da funcionalidade de cadastro de empresas com o @peterlima.

    @melissawen hoje ja é assim, ele salva em branco, acho que isso será resolvido 100% com a evolução da funcionalidade, possibilitando o cadastro caso não exista.

    eu e o @peterlima e a @marisa haviamos conversado no ministério e vamos incluir a evolução com o cadastro da empresa caso ela não exista na lista.

    O primeiro passo para isso é ver com o @peterlima e @melissawen a melhor iteração/design para isso.

    Logo eu vou fechar essa discução já que apenas empersas cadastradas entram no relato, e que a necessidade é a evolução permitindo o cadastro e podemos discutir isso na issue específica sobre essa evolução.

    Caso achem necessário podem reabrir essa issue.

    Essa issue foi mapeada para a issue https://softwarepublico.gov.br/gitlab/softwarepublico/softwarepublico/issues/327

    #327

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

    Status changed to closed

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

    @melissawen o problema de salvar a empresa com o que o usuário informar é que este cadastro possui outros campos como obrigatórios. Talvez o caminho seja deixar como obrigatório mas tem o botão para ele chamar o cadastro de empresa à parte... @thiago para conhecimento.

    Choose File ...   File name...
    Cancel
  • 8646c9570ca7b4ae286a739780af0bdd?s=40&d=identicon
    Joenio Costa @joenio

    Milestone changed to [Noosfero] Evolução do Relato de Uso

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

    Milestone removed

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

    Reassigned to @marisa

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

    Tem alguma coisa estranha entre o catálogo de instituições e as instituições que aparecem no campo de instituições do relato. Está dando divergência de dados. É necessário investigar. Imagens em anexo.

    Outra coisa é que está permitindo o cadastro do relato sem empresa mas com valores e pessoas beneficiadas o que está incorreto. O correto é o usuário selecionar a empresa e depois preencher os campos numéricos. Não deveria existir relato com valores mas e empresa associada, isso é um erro.

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

    Mais uma

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

    Status changed to reopened

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

    Reassigned to @thiago

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

    @marisa em relação a divergência do catalogo de instituições e das intituições disponíveis no portal, eu vou criar uma issue específica pra esse problema.

    #700

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

    @marisa em relação a outra observação, fizemos uma validação e agora ficaria assim:

    • preencheu apenas comentarios: é possível salvar

    • preencheu comentarios e instituiçãos: é possível salvar

    • preencheu cometarios instituições e valores: é possível salvar

    • preencheu comentarios e valores dos recursos: NÃO é possível salvar.

    Eu fiquei em dúvida em relação ao caso de preencher o comentário e a instituição e não preencher os valores, se é um relato válido ou não?

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

    Isso. Os valores só são válidos com instituição associada, sem instituição não faz sentido, no entanto o contrário pode ser verdadeiro, ele pode informar a instituição e não desejar informar os valores, sem problemas.

    Choose File ...   File name...
    Cancel
  • Df604377056e7cd36436e90bf2b9af70?s=40&d=identicon
    Gabriel Silva @gabrielssilva

    mentioned in merge request !170

    Choose File ...   File name...
    Cancel
  • 2fb770603e9880964ce0f8ac999d516f?s=40&d=identicon
    Rodrigo Souto @diguliu
    Choose File ...   File name...
    Cancel
  • 800e3af93350753db9dee1864cef10ff?s=40&d=identicon
    Thiago Ribeiro @thiago

    Pessoal, a princípio o erro foi corrigido. Podem validar? Obrigado!

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

    Status changed to closed

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

    Reabrindo a issue, pois o campo de nome da instituição não auto completa quando o usuário não é admin

    @tallysmartins confirmou comigo hoje a tarde

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

    Status changed to reopened

    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
  • Cae6de821da30189b2573c26aa4ddfa1?s=40&d=identicon
    Tallys Martins @tallysmartins

    Status changed to closed

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

    @lmatos verifiquei aqui e o problema já parece estar corrigido. Vou fechar a issue para controle interno da equipe.

    Att

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

    mentioned in issue #855

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

    mentioned in issue #855

    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

    Está aceitando qualquer texto no campo empresa.

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

    @marcosronaldo : ajudar com isso, por favor; @kanashiro : acompanhar, por favor.

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

    @marisa assim como o que está em produção, o campo faz a busca pelas instituições/empresa mas não impede que o usuário entre com um texto qualquer. Porém se o usuário preencher uma texto aleatório e for uma empresa inexistente, essa informação do relato é descartada.

    O último problema que havia sido apontado pelo @lmatos era no campo que não auto completava para usuários não admins e por isso não era possível fazer relato indicando valores economizados e pessoas, pois o relato falha se não tiver a empresa associada.

    @marisa houve problema na homologação do comportamento acordado anteriormente?

    • preencheu apenas comentarios: é possível salvar

    • preencheu comentarios e instituiçãos: é possível salvar

    • preencheu cometarios instituições e valores: é possível salvar

    • preencheu comentarios e valores dos recursos: NÃO é possível salvar.

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

    Isso está associado ao erro do fluxo que não permite que usuário comum cadastre uma instituição. Corrigindo o fluxo acreditamos que resolve. A outra questão é que o usuário até não consegue cadastrar mas a mensagem de erro informada ao não está tratada. (Desculpe, ocorreram problemas na avaliação deste perfil!)

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

    Reassigned to @marcosronaldo

    Choose File ...   File name...
    Cancel
  • C8b72d0556872a2aea21e8fed0a72001?s=40&d=identicon
    Melissa Wen @melissawen

    @marcosronaldo, sobre a mensagem de erro, eu e o @peterlima sugerimos que seja tratada como proposto em #862

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

    Já é possível que usuários comuns cadastrem instituições normalmente.

    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

    Podemos homologar em dev?

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

    @marisa pode verificar em produção.

    Choose File ...   File name...
    Cancel