Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refatorar uso do formulário RasaConversationForm, seguindo a convenção do django. #358

Open
4 tasks
davidCarlos opened this issue Jul 14, 2021 · 0 comments
Assignees
Labels
ej-server Correções e features na aplicação django prioridade::baixa Nós iremos trabalhar nessa issue se possível, havendo a possibilidade de replanejamento refatoração

Comments

@davidCarlos
Copy link

A implementação atual do form sobrescreve o método init, mas essa estratégia quebra a convenção do Django para forms que interagem com Models e o banco.

https://docs.djangoproject.com/en/3.2/topics/forms/modelforms/

Tarefas

  • Refatorar tempate jinja, para que inputs tenham o mesmo nome que os campos do form;
  • Refatorar rota para passar como argumento para o form o request.POST
  • Remover sobrescrita do método init, dentro do form;
  • Corrigir os testes
@davidCarlos davidCarlos added ej-server Correções e features na aplicação django prioridade::media Nós iremos trabalhar nessa issue assim que possível, caso não haja nenhum bug ou feature crítica refatoração labels Jul 14, 2021
@davidCarlos davidCarlos added prioridade::baixa Nós iremos trabalhar nessa issue se possível, havendo a possibilidade de replanejamento and removed prioridade::media Nós iremos trabalhar nessa issue assim que possível, caso não haja nenhum bug ou feature crítica labels Jul 20, 2021
@davidCarlos davidCarlos modified the milestone: demo.eco 0.13 Jul 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ej-server Correções e features na aplicação django prioridade::baixa Nós iremos trabalhar nessa issue se possível, havendo a possibilidade de replanejamento refatoração
Projects
None yet
Development

No branches or pull requests

2 participants