Bootcamp: Implementando CBV

Criado em 30 mai. 2016  ·  11Comentários  ·  Fonte: vitorfs/bootcamp

Isso exigiria uma grande quantidade de trabalho para refatorar o código neste nível, dado o fato de que todas as visualizações já funcionam sem problemas. Mas isso deve reduzir a quantidade de código real e implementar práticas mais reais e desejáveis ​​no desenvolvimento do projeto.

enhancement help wanted

Todos 11 comentários

alguém trabalhando nisso?

Olá, @anweshknayak Não agora. Você está interessado em trabalhar nisso?

@ sebastian-code Sim, estou. Deixe-me configurar este projeto. Eu vou te perguntar sobre qualquer dúvida

Ok, @anweshknayak , eu tentaria o meu melhor para ajudar.

@ sebastian-code devo fazer um PR diferente para cada aplicativo?

Olá @anweshknayak , acho que é a melhor abordagem.

@sebastian-code você pode dar algumas dicas sobre que tipo de visualização usar? (https://ccbv.co.uk/) Acho que podemos usar DetailView ou ListView?
Começando aqui (https://github.com/vitorfs/bootcamp/blob/master/bootcamp/activities/views.py#L10)

Olá @anweshknayak, isso seria uma lista de objetos (notificações), então a abordagem certa seria herdar de ListView, também, não se esqueça de herdar do mixin nativo LogginRequired para o modo de exibição.

@ sebastian-code, deseja que cada visão seja reescrita como cbv?

Bem, não necessariamente @tamirOK CBV visam principalmente reduzir o detalhamento e evitar casos de _DRY_, então eu imagino que em alguns casos faz mais sentido ter uma visão funcional, e isso requer a aplicação de lógica para evitar trabalho desnecessário. Eu diria que deixo para você se você quer dar algo ao projeto.

Qualquer ajuda é apreciada.

Mas não esqueça de adicionar teste, um dos objetivos é melhorar o percentual de cobertura.

Fechar este problema porque é muito amplo e inespecífico, o que o torna inútil. E porque é uma meta já incluída no # 66, então podemos mantê-la lá.

Esta página foi útil?
0 / 5 - 0 avaliações

Questões relacionadas

norouzzadeh picture norouzzadeh  ·  3Comentários

mwanjajoel picture mwanjajoel  ·  5Comentários

yashLadha picture yashLadha  ·  21Comentários

sebastian-code picture sebastian-code  ·  11Comentários

phamminhtris picture phamminhtris  ·  12Comentários