Plots2: إخفاء "اختبار بريد الملخص" من الإنتاج

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

مصنوع من # 4280

مشكلة

يجب ألا يظهر الزر Test Digest Mail في خادم الإنتاج.

screenshot 113

رابط الكود

https://github.com/publiclab/plots2/blob/58d5ea84462f6117ce55830aeb8657eca3e5ac7a/app/views/users/profile.html.erb#L237 -L241

bug

ال 10 كومينتر

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

مرسلة مع GitHawk

قد يكون باستطاعة sidharthbansal و gauravano تقديم المساعدة هنا ..

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

أعلم أنه للاختبار ولكن يمكنك تغيير نمط الزر لمطابقة الأزرار الأخرى.

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

gauravano لذا يجب أن يظهر في الإنتاج إذا كان المستخدم مشرفًا أو مشرفًا؟

مرسلة مع GitHawk

بالطريقة الحالية ، يظهر الزر فقط عندما لا يكون في مرحلة الإنتاج ، ولا يأخذ حالة المستخدم في الاعتبار.

مرسلة مع GitHawk

نعم uzorjchibuzor لا بأس بإظهار الزر في الإنتاج للمشرفين والمشرفين.

uzorjchibuzor هل ترغب في العمل على هذه المشكلة؟
إذا كانت الإجابة بنعم ، فاستمر!

uzorjchibuzor هل ترغب في العمل على هذه المشكلة؟ ...

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

مرسلة مع GitHawk

اوه رائع..

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