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.
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á.