Ini akan membutuhkan banyak pekerjaan untuk memperbaiki kode pada tingkat ini, mengingat fakta bahwa semua tampilan sudah berfungsi dengan lancar. Tetapi ini harus mengurangi jumlah kode aktual dan akan menerapkan praktik yang lebih aktual dan diinginkan pada pengembangan proyek.
sebastian-code
Semua 11 komentar
ada yang mengerjakan ini?
anweshknayak
pada 9 Jan 2017
Hai @anweshknayak Tidak sekarang. Apakah Anda tertarik untuk mengerjakan ini?
sebastian-code
pada 9 Jan 2017
@sebastian-code Ya saya. Biarkan saya mengatur proyek ini. Saya akan bertanya kepada Anda atas pertanyaan apa pun
anweshknayak
pada 9 Jan 2017
Ok @anweshknayak , saya akan mencoba yang terbaik untuk membantu.
sebastian-code
pada 9 Jan 2017
@sebastian-code haruskah saya membuat PR yang berbeda untuk setiap aplikasi?
anweshknayak
pada 14 Jan 2017
Hai @anweshknayak, menurut saya, ini adalah pendekatan terbaik.
sebastian-code
pada 14 Jan 2017
@sebastian-code dapatkah Anda memberikan petunjuk tentang tampilan seperti apa yang digunakan? ( https://ccbv.co.uk/) Saya pikir kita bisa menggunakan DetailView atau ListView ?
Mulai dari sini (https://github.com/vitorfs/bootcamp/blob/master/bootcamp/activities/views.py#L10)
anweshknayak
pada 16 Jan 2017
Hai @anweshknayak itu akan menjadi daftar objek (pemberitahuan) sehingga pendekatan yang tepat adalah mewarisi dari ListView, juga, jangan lupa untuk mewarisi dari mixin LogginRequired asli ke dalam tampilan.
sebastian-code
pada 17 Jan 2017
@sebastian-code, apakah Anda ingin setiap tampilan ditulis ulang sebagai cbv?
tamirOK
pada 10 Mar 2017
Yah, belum tentu @tamirOK CBV terutama ditujukan untuk mengurangi verbositas dan untuk menghindari kasus _DRY_, jadi saya membayangkan daripada dalam beberapa kasus lebih masuk akal untuk memiliki tampilan fungsional, dan itu perlu menerapkan logika untuk menghindari pekerjaan yang tidak perlu. Saya akan mengatakan daripada saya menyerahkannya kepada Anda apakah Anda ingin memberikan sesuatu untuk proyek tersebut.
Bantuan apa pun dihargai.
Tapi jangan lupa untuk menambahkan tes, salah satu tujuannya adalah untuk meningkatkan persentase cakupan.
sebastian-code
pada 10 Mar 2017
Menutup masalah ini karena terlalu luas dan tidak spesifik, yang membuatnya tidak membantu. Dan karena gol sudah termasuk dalam #66 jadi kami bisa mengikutinya di sana.
sebastian-code
pada 13 Mei 2017
Masalah terkait
Allan-Nava
·
9Komentar
Shekharnunia
·
10Komentar
norouzzadeh
·
3Komentar
Allan-Nava
·
14Komentar
ssahilsahil798
·
5Komentar