Plots2: انقل المساعد comment_render_body إلى نموذج التعليق

تم إنشاؤها على ١ أكتوبر ٢٠١٨  ·  4تعليقات  ·  مصدر: publiclab/plots2

تم تصنيف هذا كمرشح جيد ليصبح مشكلة لأول مرة فقط مثل هذه ، مما يعني أنه بسيط ومكتفي بذاته ومع بعض التنسيق الإضافي ، يمكن أن يكون نقطة دخول رائعة لمساهم جديد. إذا كنت على دراية كافية بهذا الرمز ، فالرجاء التفكير في إعادة تنسيقه أو إعادة نشره كمشكلة لأول مرة فقط ، ثم ping @ publiclab / المراجعين للحصول عليها. أو ، إذا لم تكن هذه هي المرة الأولى ، فحاول حلها بنفسك!

https://github.com/publiclab/plots2/blob/6595ef39f25ca4f2d377cacfd92e86a7460214d6/app/helpers/application_helper.rb#L100 -L107

لنجعل هذه طريقة للتعليق - في https://github.com/publiclab/plots2/blob/master/app/models/comment.rb - مثل:

def render_body

end

وإزالته من application_helper.rb !

سنحتاج أيضًا إلى تغيير هذا السطر:

https://github.com/publiclab/plots2/blob/6595ef39f25ca4f2d377cacfd92e86a7460214d6/app/views/notes/_comment.html.erb#L120

إلى comment.render_body

أخيرًا ، يمكن تحديث اختبار sanitize comment body في https://github.com/publiclab/plots2/blob/master/test/unit/comment_test.rb لاختبار comment.render_body بدلاً من إعادة إنتاج الكود من المساعد.

سعيد للمساعدة في هذا واحد! يمكن أن تكون مشكلة لأول مرة!

Ruby fto-candidate help wanted refactorization
مصدر
picture jywarren

التعليق الأكثر فائدة

أود أن أتناول هذه المسألة ، ستكون واحدة من أولى مساهماتي مفتوحة المصدر. هل هناك أي شيء أحتاج إلى معرفته قبل فتح طلب سحب ، مثل مخطط خاص لرسائل الالتزام؟

picture mauricetmeyer في ١ أكتوبر ٢٠١٨
🎉1 👍1

ال 4 كومينتر

أود أن أتناول هذه المسألة ، ستكون واحدة من أولى مساهماتي مفتوحة المصدر. هل هناك أي شيء أحتاج إلى معرفته قبل فتح طلب سحب ، مثل مخطط خاص لرسائل الالتزام؟

mauricetmeyer picture mauricetmeyer في ١ أكتوبر ٢٠١٨
🎉1 👍1

لا ، هناك نموذج في نموذج طلب السحب ، لكنه قابل للتعديل دائمًا لاحقًا ، لذا يمكننا مساعدتك كما تذهب! أهلا بك! 👍 🎉

jywarren picture jywarren في ١ أكتوبر ٢٠١٨
👍1

@ jywarren شكرا لك! تعد أيضًا مشكلات المؤقت هذه رائعة للتعرف على قاعدة الشفرة ، حيث لا أجد في كثير من الأحيان الكثير من الوقت للقراءة في قواعد التعليمات البرمجية الكبيرة. 👍

mauricetmeyer picture mauricetmeyer في ٢ أكتوبر ٢٠١٨

سعيدة للغاية وكنت مثلهم!!!

الإثنين، 1 أكتوبر 2018 في 18:09 موريس T. ماير [email protected]
كتب:

jywarren https://github.com/jywarren شكرا لك! أيضا تلك أول جهاز توقيت
تعد المشكلات أمرًا رائعًا للتعرف على قاعدة التعليمات البرمجية ، حيث لا أمتلك الكثير في كثير من الأحيان
حان الوقت للقراءة في قواعد بيانات كبيرة. 👍

-
أنت تتلقى هذا لأنه تم ذكرك.
قم بالرد على هذا البريد الإلكتروني مباشرة ، وقم بعرضه على GitHub
https://github.com/publiclab/plots2/issues/3552#issuecomment-426080866 ،
أو كتم الخيط
https://github.com/notifications/unsubscribe-auth/AABfJ7XhfzVY7CxWNvf58mMmOiKBNZjlks5ugpKSgaJpZM4XCx6-
.

jywarren picture jywarren في ٢ أكتوبر ٢٠١٨
هل كانت هذه الصفحة مفيدة؟
0 / 5 - 0 التقييمات

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

jywarren picture jywarren  ·  3تعليقات
bronwen9 picture bronwen9  ·  3تعليقات
milaaraujo picture milaaraujo  ·  3تعليقات
ebarry picture ebarry  ·  3تعليقات
grvsachdeva picture grvsachdeva  ·  3تعليقات