ربط المواضيع بشكل اختياري بالموضوع الأصلي في تكامل Slack

لقد قمت بتنظيف طلب السحب (PR) بشكل كبير، لكنه ليس جاهزًا للإرسال بعد. لديّ حاليًا من اثنين إلى ثلاثة أشياء تسبب لي مشكلة ولا أعرف بعد كيفية إصلاحها.

  1. أحاول استخدام fa-arrow-circle-o-right كأيقونة للموضوع، وهي تظهر فارغة في واجهة المستخدم على موقعي المباشر. (لقد قمت بتشغيل su discourse -c 'bundle exec rake assets:precompile' && sv restart unicorn بعد سحب فرعتي على موقعي المباشر للاختبار على الخادم المباشر.) لقد أضفتها إلى ملف plugin.rb بالإضافة إلى الإشارة إليها، لذا أنا حائر بشأن الخطوات التالية. هل توجد قائمة بأيقونات FontAwesome المعتمدة للاستخدام في Discourse؟ وجدت lib/svg_sprite/svg_sprite.rb، وchevron-right تبدو رائعة لهذا الغرض.

  2. جميع الاختبارات تنجح لدي محليًا، لكن في Travis أحصل على أخطاء متسقة تبدو غير مرتبطة بتغييراتي، وهذا بطبيعته صعب التحقيق أو التحليل بالنسبة لي. 13 فشلًا مع خطأ 404 بدلاً من بعض الأخطاء المتوقعة الأخرى (مثل 200) في ملف spec/lib/discourse_chat/provider/slack/slack_command_controller_spec.rb تم الإصلاح بعدم استخدام isolate_namespace بشكل عشوائي، والآن أصبحت أعرف عن أمر rake routes.

لقد نجحت في النشر:

قد يكون هناك المزيد من التنظيف المطلوب، لكنني أعتقد أن هذا يعمل.

بعد دمج هذا الطلب، سأقوم بتحديث Discourse Chat Integration بشكل مناسب.

إعجابَين (2)