search

FiquemSabendo / queremossaber

MIT License
39 stars 7 forks source link

Cadastro de órgão federal ou por região #4

Closed ltartari closed 5 years ago

ltartari commented 5 years ago

Permitir cadastro de órgão federal ou de região específica.

Exemplos: Tribunal Superior Eleitoral (http://www.tse.jus.br/), Tribunal Regional Federal da 4ª Região (https://www.trf4.jus.br/trf4/).

itepifanio commented 5 years ago

Como seria esse fluxo? Cadastro simples por qualquer usuário, ou após submetido o orgão precisa ser aceito por alguém?

ltartari commented 5 years ago

Boa pergunta, @itepifanio 🤔

O fluxo pode permanecer igual ao que já existe implementado para criação de órgão… vejo que isso seria apenas uma nova opção para selecionar órgão federal ou região (não atrelado a nenhuma UF específica).

Acredito que o sistema não faça moderação automática (certo, @vitorbaptista ?). Se houver moderação o ideal seria isso ocorrer após a criação do órgão/pedido (só pra evitar de travar uma pessoa que esteja adicionando um órgão para fazer um novo pedido).

Faz sentido?

itepifanio commented 5 years ago

Entendi sim, posso trabalhar nessa issue?

vitorbaptista commented 5 years ago

Putz! Desculpem pela demora.

Não há moderação automática.

Com relação a solução, imagino adicionar um campo "Esfera" (em inglês, "level") depois do campo nome, podendo ser:

Por padrão, no formulário está "Municipal".

Daí, com JS desabilitamos o Estado/Cidade, dependendo da esfera escolhida (desabilitar, não esconder). Caso seja Estadual, exige Estado, caso Municipal, exige Estado e Cidade, e caso Federal, nada.

Também é importante fazer uma migração dos dados atuais, pra popular a esfera do que a gente já tem.

@itepifanio Você tá com tempo pra pegar essa issue? :)

itepifanio commented 5 years ago

Entendi, posso fazer no fim de semana (meu período acaba na quinta), tudo bem?

vitorbaptista commented 5 years ago

@itepifanio Maravilha! Se algo não tiver ficado claro, ou você tiver qualquer outra dúvida, só mandar uma mensagem.

itepifanio commented 5 years ago

@vitorbaptista a parte mais simples está pronta, só não entendi essa migração que você falou, seria verificar a esfera dos orgãos cadastrados anteriormente?

Outra coisa, o disabled ou readonly do materialize é bem discreto pro usuário, só evita dele clicar lá, mas visualmente não tem muitas alterações.

vitorbaptista commented 5 years ago

Maravilha!

@itepifanio Sobre as migrações, me refiro a duas coisas:

  1. Se você mudou as tabelas do banco, o que acontece com os dados que já estavam lá? Por exemplo, se criou um novo campo level (esfera) pra o PublicBody, e os public bodies que já estão no BD, vão ficar com esse campo vazio? É importante fazer uma migração pra popular isso.

  2. Atualizar as fixtures que temos em https://github.com/okfn-brasil/pedidosanonimos/blob/7880952dbc4d7236b7af9a28a70ba58e4eb9e9c6/web/foi_requests/fixtures/public_bodies_and_esics.json.

Manda um pull request com o código que você tem que fica mais fácil da gente conversar.

itepifanio commented 5 years ago

Entendi, do jeito que fiz adicionei um "default" no model, então o load_fixtures já fica com o valor padrão quando roda, caso as próximas sejam diferentes basta adicionar no json um 'level' : 'outro valor' que será preenchido outro valor.

(Daqui a pouco vejo o erro de integração do pull)