Resolve #4 - Githubissues

okfn-brasil / pedidosanonimos

MIT License

39 stars 7 forks source link

Resolve #4 #7

Closed itepifanio closed 5 years ago

itepifanio commented 5 years ago

Adiciona campo "esfera" ao formulário de cadastro de orgão público

vitorbaptista commented 5 years ago

Oi @itepifanio, adicionei mais alguns comentários triviais. Fora isso, os testes estão quebrando (https://travis-ci.org/okfn-brasil/pedidosanonimos/builds/482108579), acho que porque faltou gerar as migrações. Se for, só rodar python manage.py makemigrations -n add_public_body_level e já era :tada:

vitorbaptista commented 5 years ago

Oi @itepifanio, você ainda tem um tempinho pra ver os meus últimos comentários acima? Sem problemas se não, você já fez 99.9% do código, eu posso finalizar os 0.1% restantes.

itepifanio commented 5 years ago

@vitorbaptista deixa só eu dar o último commit, foi mal a demora, é que recebi uma outra demanda de outro projeto, mas final de semana eu subo as observações que você fez

vitorbaptista commented 5 years ago

@itepifanio Merged!

Tomei a liberdade de fazer algumas alterações. Em resumo, eu:

Adicionei alguns testes
Corrigi um bug que fazia os inputs não estarem desabilitados no load inicial (por exemplo, quando a validação falha)
Corrigi o teste que estava falhando

Muito obrigado pela contribuição! :tada: