Plots2: कमेंट_रेंडर_बॉडी हेल्पर को कमेंट मॉडल में ले जाएं

को निर्मित 1 अक्तू॰ 2018 · 4टिप्पणियाँ · स्रोत: publiclab/plots2

यह एक बनने के लिए एक अच्छे उम्मीदवार के रूप में चिह्नित किया गया है पहले टाइमर-केवल मुद्दे की तरह इन , जिसका अर्थ है कि यह सरल, आत्म निहित है, और कुछ अतिरिक्त स्वरूपण के साथ, एक नया योगदान के लिए एक महान प्रवेश बिंदु हो सकता है। यदि आप इस कोड से पर्याप्त रूप से परिचित हैं, तो कृपया इसे केवल-पहली बार के मुद्दे के रूप में पुन: स्वरूपित करने या फिर से पोस्ट करने पर विचार करें, और फिर इसे लेबल करने के लिए @publiclab/reviewers को पिंग करें। या, यदि यह आपका पहली बार नहीं है, तो इसे स्वयं हल करने का प्रयास करें!

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

अंत में, https://github.com/publiclab/plots2/blob/master/test/unit/comment_test.rb में sanitize comment body परीक्षण को कोड को पुन: प्रस्तुत करने के बजाय comment.render_body का परीक्षण करने के लिए अद्यतन किया जा सकता है सहायक से।

इस के साथ मदद करने में खुशी हुई! पहली बार का मुद्दा हो सकता है!

Ruby fto-candidate help wanted refactorization

स्रोत

jywarren

सबसे उपयोगी टिप्पणी

मैं इस मुद्दे को उठाना चाहता हूं, यह मेरे पहले ओपन सोर्स योगदानों में से एक होगा। पुल अनुरोध खोलने से पहले मुझे कुछ भी जानने की आवश्यकता होगी, जैसे प्रतिबद्ध संदेशों के लिए एक विशेष स्कीमा?

mauricetmeyer 1 अक्तू॰ 2018

🎉1 👍1

सभी 4 टिप्पणियाँ

mauricetmeyer 1 अक्तू॰ 2018

🎉1 👍1

नहीं, पुल अनुरोध फ़ॉर्म में एक टेम्प्लेट है, लेकिन इसे बाद में हमेशा संपादित किया जा सकता है, इसलिए जैसे ही आप जाते हैं हम आपकी मदद कर सकते हैं! स्वागत! मैं

jywarren 1 अक्तू॰ 2018

👍1

@jywarren धन्यवाद! कोडबेस को जानने के लिए पहले टाइमर के मुद्दे भी बहुत अच्छे हैं, क्योंकि मेरे पास अक्सर बड़े कोडबेस में पढ़ने के लिए ज्यादा समय नहीं होता है। मैं

mauricetmeyer 2 अक्तू॰ 2018

बहुत खुशी है कि आप उन्हें पसंद करते हैं !!!

सोम, 1 अक्टूबर, 2018 को शाम 6:09 बजे मौरिस टी। मेयर सूचनाएं @github.com
लिखा था:

@jywarren https://github.com/jywarren धन्यवाद! वो भी पहले टाइमर
कोडबेस को जानने के लिए मुद्दे बहुत अच्छे हैं, क्योंकि मेरे पास अक्सर बहुत कुछ नहीं होता है
बड़े कोडबेस में पढ़ने का समय। मैं
-
आप इसे प्राप्त कर रहे हैं क्योंकि आपका उल्लेख किया गया था।
इस ईमेल का सीधे उत्तर दें, इसे GitHub पर देखें
https://github.com/publiclab/plots2/issues/3552#issuecomment-426080866 ,
या थ्रेड को म्यूट करें
https://github.com/notifications/unsubscribe-auth/AABfJ7XhfzVY7CxWNvf58mMmOiKBNZjlks5ugpKSgaJpZM4XCx6-
.

jywarren 2 अक्तू॰ 2018

क्या यह पृष्ठ उपयोगी था?

0 / 5 - 0 रेटिंग्स

Plots2: कमेंट_रेंडर_बॉडी हेल्पर को कमेंट मॉडल में ले जाएं

सबसे उपयोगी टिप्पणी

सभी 4 टिप्पणियाँ

संबंधित मुद्दों