Bootcamp: تنفيذ CBV

تم إنشاؤها على ٣٠ مايو ٢٠١٦  ·  11تعليقات  ·  مصدر: vitorfs/bootcamp

سيتطلب هذا قدرًا كبيرًا من العمل لإعادة تشكيل الكود على هذا المستوى ، نظرًا لأن جميع طرق العرض تعمل بالفعل بسلاسة. ولكن هذا من شأنه أن يقلل من مقدار الكود الفعلي وأن ينفذ المزيد من الممارسات الفعلية والمرغوبة في تطوير المشروع.

enhancement help wanted

ال 11 كومينتر

أي شخص يعمل على هذا؟

مرحبًا anweshknayak ليس الآن. هل أنت مهتم بالعمل على هذا؟

@ sebastian-code نعم أنا. اسمحوا لي أن أقوم بإعداد هذا المشروع. سوف أسألك عن أي استفسار

حسنًا anweshknayak ، سأبذل قصارى جهدي للمساعدة.

@ sebastian-code هل يجب أن أقوم بعمل علاقات عامة مختلفة لكل تطبيق؟

مرحبًا anweshknayak أعتقد أنه أفضل نهج.

@ sebastian-code هل يمكنك إعطاء بعض التلميح عن نوع العرض الذي يجب استخدامه؟ (https://ccbv.co.uk/) أعتقد أنه يمكننا استخدام DetailView أو ListView؟
بدءًا من هنا (https://github.com/vitorfs/bootcamp/blob/master/bootcamp/activities/views.py#L10)

مرحبًا anweshknayak من شأنها أن تكون قائمة بالكائنات (الإخطارات) ، لذا فإن الطريقة الصحيحة هي أن ترث من ListView ، أيضًا ، لا تنس أن ترث مزيج LogginRequired الأصلي في العرض.

@ sebastian-code ، هل تريد إعادة كتابة كل عرض على هيئة cbv؟

حسنًا ، ليس بالضرورة أن تهدفtamirOK CBV بشكل أساسي إلى تقليل الإسهاب وتجنب حالات _DRY_ ، لذلك أتخيل أنه في بعض الحالات يكون من المنطقي أن يكون لديك وجهة نظر وظيفية ، وهذا يتطلب تطبيق المنطق لتجنب العمل غير الضروري. أود أن أقول أكثر مما أتركه لك هل تريد أن تعطي شيئًا للمشروع.

هو موضع تقدير أي مساعدة.

لكن لا تنس إضافة اختبار ، أحد الأهداف هو تحسين نسبة التغطية.

إغلاق هذه القضية لأنها واسعة وغير محددة مما يجعلها غير مفيدة. ولأنه هدف مُدرج بالفعل في رقم 66 حتى نتمكن من مواكبة ذلك هناك.

هل كانت هذه الصفحة مفيدة؟
0 / 5 - 0 التقييمات

القضايا ذات الصلة

Shekharnunia picture Shekharnunia  ·  10تعليقات

Allan-Nava picture Allan-Nava  ·  9تعليقات

sebastian-code picture sebastian-code  ·  11تعليقات

Shekharnunia picture Shekharnunia  ·  6تعليقات

phamminhtris picture phamminhtris  ·  12تعليقات