بالنسبة للمراحل اللاحقة من مشروع محرر التعليقات ، عندما يتم تحويله إلى مستودع خاص به ، قد نرغب في التفكير بعناية في سير العمل هذا الموثق بواسطة Sagarpreet لما قد يساعد في إنشاء دورة إصدار قياسية!

https://publiclab.org/notes/sagarpreet_chadha/10-20-2020/first-timer-only-release-workflow

مع ملاحظة هذا المثال لمربعات التعليق عبر الأسلاك أثناء تحميل الصورة إلى تعليق يتم تحريره (لم تتم كتابته حديثًا): https://github.com/publiclab/plots2/issues/8670

تم نقل مشكلة التخطيط هنا -> # 9069

لقد قررت نقل مشكلة التخطيط الخاصة بي إلى صفحة إصدار جديدة ، في الغالب لقضايا سهولة القراءة. آسف على أي لبس يسببه هذا! - @ noi5e

مرحبًا @ noi5e ، تبدو مشكلة التخطيط رائعة.
ما هي القضية التي تعمل عليها حاليا؟ هل يمكنك أن تدلني على العلاقات العامة إن وجدت ، أخبرني أيضًا إذا كنت عالقًا في مكان ما أو تحتاج إلى بعض الوضوح. شكرا 😄

@ sagarpreet-chadha شكرًا لك على تسجيل الوصول معي! منذ نشر هذا ، كنت أكتب منشور المدونة الخاص بي لـ Outreachy (في أجزاء وأجزاء ، على وشك الانتهاء) ، وأجري أبحاثًا وأعمل على علاقات عامة لـ 8618 ، وأقوم بتنظيف موقع Rails - والذي ليس لدي خلفية قوية فيه ، ولكني كنت كذلك تعلم الكثير هذا الأسبوع. عملت لمدة يومين في وظيفتي الأخرى ، لذلك أشعر بتأخر قليل ، لكنني قمت بالكثير من اللحاق بالركب اليوم وسأبذل المزيد غدًا والاثنين. أخطط لكتابة الاختبارات للتعليقات غدا.

أحتاج إلى القليل من التوضيح حول اختبار التعليقات. لقد ألقيت نظرة على /plots2/blob/main/test/system/comment_test.rb ويبدو أن هناك بالفعل اختبارات لإدخال التعليقات يدويًا وعبر عنوان URL الخاص بواجهة برمجة التطبيقات ، وكلاهما يشير إلى /wiki/wiki-page-path/comments .

إذا كنت سأكتب اختبارات تعليق جديدة لصفحات الأسئلة والملاحظات ، فهل سيكون عنوان URL مختلفًا لواجهة برمجة التطبيقات أم هو نفسه كما هو مذكور أعلاه ( wiki-page-path )؟ ركضت على rails routes لكن ليس من الواضح بالنسبة لي أيها من المفترض أن أختبرها. شكرًا مقدمًا ، سيوفر هذا بعض الوقت في البحث عن الشفرة. 😅

تحرير: أخبرني أيضًا إذا كان لديك طلبات محددة لأنواع الاختبارات التي يجب أن أكتبها!

مرحبًا @ noi5e ، أعتقد أن مسار "تعليقات wiki" يختلف عن مسار "الملاحظات" - كل من الويكي والملاحظات هي متغيرات من Node - ولكن يجب أن تكون متشابهة جدًا.

لقد مر وقت طويل منذ أن نظرت في هذا المجال من التعليمات البرمجية ، ولكن عادةً ما يكون لدى ريلز إجراء تحكم لكل مسار ، ولكن الاستثناء هو عندما تكون مسارات حيلة :

مشاهدة هذا الإجراء لعرض التعليقات:

https://github.com/publiclab/plots2/blob/cbb807ba8e2302f09dafc0060475aa118e34c2c6/app/controllers/wiki_controller.rb#L467 -L470

يبدو أنه لا يوجد إجراء تحكم للنشر. لكني أرى أن هذا الطريق:

https://github.com/publiclab/plots2/blob/cbb807ba8e2302f09dafc0060475aa118e34c2c6/config/routes.rb#L58

يشير إلى هذا الإجراء الرأي. إذن أين طرق الخلق وأفعاله؟ أرى البعض هنا ، ليس خاصًا بـ wiki على الرغم من:

https://github.com/publiclab/plots2/blob/cbb807ba8e2302f09dafc0060475aa118e34c2c6/config/routes.rb#L368 -L373

لذلك دعونا نلقي نظرة على الطرف الآخر من الأشياء ، القوالب. هنا نموذج التعليق الأساسي:

https://github.com/publiclab/plots2/blob/main/app/views/comments/_form.html.erb

أعتقد أن التعليقات على الأسئلة قد تعيد استخدام هذا النموذج ، بناءً على المنطق الموجود في علامة <form> هنا؟

https://github.com/publiclab/plots2/blob/cbb807ba8e2302f09dafc0060475aa118e34c2c6/app/views/comments/_form.html.erb#L3

لذلك ، يشيرون إلى /comments/create/NID باستخدام طريقة POST ، وبالتالي فإن المسار الأخير من السطر 373 من المسارات. تلك النقاط في وحدة التحكم في التعليقات create action:

https://github.com/publiclab/plots2/blob/cbb807ba8e2302f09dafc0060475aa118e34c2c6/app/controllers/comment_controller.rb#L19 -L56

لذلك أعتقد أنه من الآمن القول أن معظم التعليقات تستخدم إجراء التحكم في التعليقات create . ولكن ماذا عن اختبارات النظام التي لم يتم ذكر هذا الطريق فيها؟

الطريقة الأخرى التي تحدث هنا هي عبر طريقة JavaScript:

https://github.com/publiclab/plots2/blob/cbb807ba8e2302f09dafc0060475aa118e34c2c6/test/system/comment_test.rb#L32

يقودنا هذا إلى ملف JS هذا ، والذي يرسله إلى نفس المسار عبر AJAX:

https://github.com/publiclab/plots2/blob/484bf69ead7a1f69c9047a25b3f63b9d6f875728/app/assets/javascripts/comment.js#L62 -L70

بالطريقة التي تبدو بها قاعدة الكود الخاصة بنا الآن ، قد تبدو الوظيفة sendFormSubmissionAjax() وكأنها لم تكن في ملفها الخاص ، لكنني أعتقد أن nstjean (شبة Outreachy رائعة ، مرحبًا !!!!!! 🎉) تم التخطيط له لتبسيط إرسال نموذج JS عبر قاعدة الرموز الخاصة بنا ، والتي أؤيدها تمامًا. ها هو!

https://github.com/publiclab/plots2/blob/876d0fc084064aaecc23f8003630d7d1ab858fa1/app/assets/javascripts/submit_form_ajax.js#L1 -L4

أتمنى أن يكون هذا مفيدًا! إنه ليس أسوأ ما رأيته من حيث تنظيم الكود ، لكنه بالتأكيد معقد. هذا جزئيًا فقط لأن هذا هو قاعدة بيانات كبيرة وقديمة شهدت العديد من مراحل المراجعة المختلفة. في الواقع ، قبل اختبارات النظام ، التي تم تثبيتها فقط في العامين الماضيين أو نحو ذلك ، لم يكن لدينا أي طريقة لإجراء اختبار متكامل لإرسال تعليقات JavaScript! لذلك كان ينكسر باستمرار :-(

شكرا يا رفاق !!!

jywarren شكرًا جزيلاً على هذه الكتابة المفيدة والمفصلة !!! لقد ساعدني حقًا في اكتشاف الكثير من الأشياء حول قاعدة البيانات. لا يزال هناك الكثير لنتعلمه. لقد قمت بعمل علاقات عامة مؤقتة للاختبار - مع الكثير من الأسئلة.

نتطلع أيضًا إلى توضيح هذه:

"حفظ واستعادة ميزة الرموز الرئيسية التي تسبب Mimi في الحل البديل" (لا تحصل على هذا)

لم أتمكن من العثور على هذا في الواقع في https://pad.publiclab.org/p/outreachy - من أين أتى؟ يبدو مألوفا بشكل غامض ... آسف!

لا تعمل المعاينة في بعض الأحيان (لا يمكنني التكاثر)

ربما يكون هذا مرشحًا جيدًا لنوع "مجموعة الاختبارات القياسية" التي يمكننا إجراؤها تلقائيًا على متغيرات متعددة لمحرر التعليقات ، عندما نعيد تنظيم كود الاختبار؟ أتساءل عما إذا كان ذلك سيؤدي إلى حدوث ذلك ، أو ربما تم حله بالفعل.

الأزرار الغامقة والمائلة "متصلة بأسلاك" (لا يمكن إعادة إنتاجها)

يبدو أن هذا موجود في https://github.com/publiclab/plots2/issues/8478 وأتساءل عما إذا كانت نفس الإستراتيجية يمكن أن تساعد ، من تضمين هذا في مجموعة قياسية من اختبارات محرر التعليقات وتشغيلها عبر جميع المتغيرات.

تأكد من وجود "السحب والإفلات لإدراج الصور" باللون الرمادي في كل مكان (لا يمكن إعادة الإنتاج ، هل ترغب في الحصول على بعض الأمثلة)

ربما كما هو مذكور أعلاه ... ولم أتمكن أيضًا من العثور عليه في https://pad.publiclab.org/p/outreachy ، لذا ربما يمكنني إضافة المزيد ، مع العلم من أين أتت؟

مشكلات إبداء الإعجاب بالتعليقات (راجع "إعجابات التعليقات لا تعمل رقم 5113"). أنا أفهم المشكلة ، فأنا في الأساس بحاجة إلى توضيح السلوك المتوقع هنا.

قد يكون هذا قديمًا حقًا ؟؟ لم يعد لدينا إعجاب بالتعليقات ، حيث تم استبداله بنظام "ردود الفعل". لكن ، ربما نوضح من خلال سؤال ساشا ، الذي صنع # 5113

شكرا @ noi5e ! أتمنى أن تساعد هذه !!!!

شكرًا jywarren بالتأكيد مفيد ... أعتقد أن النقاط التي كنت تبحث عنها موجودة في مستند Google هذا

مرحبًا @ noi5e ، لست متأكدًا مما إذا كنت قد رأيت هذا أو أصلحت هذا بالفعل ولكني لاحظت ذلك في تعليقاتنا - يبدو أن هناك شريطًا رمادي اللون غير ضروري أسفل النموذج:

أيضًا ، أعتقد أن بعض المسافات حول الرموز التعبيرية متوقفة قليلاً. شاهد كيف يبدو أنه يحتوي على حشوة بيضاء في الأعلى والأسفل ، esp عند مقارنته بأسلوب GitHub؟

فقط بضعة أشياء صغيرة ربما يمكننا معالجتها في العلاقات العامة القادمة ، لكن لا شيء عاجل. شكرا!

jywarren بالتأكيد ، لقد لاحظت هؤلاء أيضًا! إضافة إلى واجباتي.

كنت أتساءل أيضًا بعد تحسينات السرعة لـ https://github.com/publiclab/plots2/pull/9045 ، أتذكر أن Skylight تُظهر أن نشر تعليق على PublicLab.org قد يستغرق بعض الوقت ... لقد قمنا بالتحسين إلى حد ما ولكن ليس بالقدر الذي كنا نأمله. هل ما زلت تجد أنه وقت تحميل بطيء لنشر تعليق؟ أتساءل عما إذا كانت هذه أيضًا طريقة لتحسين وقت تشغيل اختبار النظام ، مع تحسين تجربة المستخدم أيضًا.

هذه لقطة. يبدو أن الأمر لا يزال يستغرق ما يصل إلى 10 ثوانٍ من حين لآخر ، ولكن للأسف ، فإن مراقبتنا ليست جيدة بما يكفي لإظهار أكثر من هذا الوقت الذي يقضيه في وحدة التحكم ، على ما يبدو:

هذا أيضًا ليس من متطلبات مشروعك ، لكنني اعتقدت أنه قد يكون ممتعًا. الكود الذي يقود هذا هنا وهنا .

لا تقلق حقًا بشأن هذا لأكون صادقًا ... لكنني أشعر بالفضول إذا شعرت أن وقت نشر التعليقات كان بطيئًا جدًا أم لا؟

jywarren نعم! أعتقد بالتأكيد أن وقت نشر التعليقات بطيء جدًا! (أيضًا ، حان الوقت للرد على تعليق)

من المثير للاهتمام معرفة المزيد عن Skylight وماذا تفعل. أتساءل ما الذي يسبب التباطؤ بالضبط ، هل تعتقد أنه قد يكون استعلام ActiveRecord؟

تحرير: إضافته إلى قائمة رغبات الأهداف الممتدة في موضوع التخطيط هذا!

تم نقل مشكلة التخطيط هنا -> # 9069

لقد قررت نقل مشكلة التخطيط الخاصة بي إلى صفحة إصدار جديدة ، في الغالب لقضايا سهولة القراءة. آسف على أي لبس يسببه هذا! - @ noi5e