Moin
5 أكتوبر 2024، 9:41م
1
شعرت بالإثارة عندما وجدت زر الإغلاق الجديد في لوحة تحكم المسؤول الخاصة بي. ومع ذلك، بعد النقر عليه، شعرت بخيبة أمل بعض الشيء. هل من المفترض أن يعمل بهذه الطريقة؟
5 إعجابات
ondrej
5 أكتوبر 2024، 10:02م
3
بدا أنه يعمل بشكل أفضل في سبتمبر
لم أواجه هذه المشكلة منذ إصدارها لأنني أتابع تحديثات TC.
3 إعجابات
ted
(Ted Johansson)
7 أكتوبر 2024، 5:15ص
5
مرحباً، @Moin !
شكراً على التقرير. بما أن رسالة الخطأ غير مفيدة على الإطلاق، فمن الصعب تحديد ما حدث بالضبط هنا، ولكنني راجعت الكود ووجدت أمرين يمكن أن يسببا خطأ:
تم رفض الإشعار بالفعل من قبل مسؤول آخر أو في علامة تبويب أخرى.
المستخدم الذي يحاول رفض الإشعار ليس مسؤولاً. (لا يمكن أن تكون هذه هي المشكلة أعلاه، لأن الخطأ مختلف.)
لقد أضفت إصلاحات لهاتين الحالتين هذا الصباح. سأترك هذا الموضوع مفتوحاً لفترة لمعرفة ما إذا كان سيتكرر.
main ← fix/concurrent-admin-notice-dismissal
opened 04:07AM - 07 Oct 24 UTC
### What is the problem?
If you have the admin dashboard open, and one of the… admin notices listed has already been dismissed (e.g. in another tab, or by another admin) we would show an ugly "FAILED" modal.
<img width="503" alt="Screenshot 2024-10-07 at 12 28 47 PM" src="https://github.com/user-attachments/assets/7f1610f8-cf4d-45d1-8b3d-81c1b4c6426d">
### How does this fix it?
Make the admin dismiss endpoint idempotent. If the admin notice is already destroyed, then respond with 200. This will also correctly remove it from the list in the front-end.
main ← fix/dont-render-dismiss-buttons-for-mods
opened 04:54AM - 07 Oct 24 UTC
### What is the problem?
Dismissing admin notices is an admin-only action. Th… is is enforced on the back-end both by a routing constraint and a policy in the relevant service.
However, we still unconditionally display the "Dismiss" button to anyone with access to the admin dashboard. When clicked, it results in a 404 modal (due to the routing constraint.)
<img width="419" alt="Screenshot 2024-10-07 at 12 54 13 PM" src="https://github.com/user-attachments/assets/1b497bf9-d067-4ded-b431-fff383f0717a">
### How does this fix it?
Only render the dismiss button for admins.
3 إعجابات
Moin
7 أكتوبر 2024، 5:20ص
6
ألن يختفي الإشعار في هذه الحالة بعد مرور بعض الوقت؟
إعجابَين (2)
ted
(Ted Johansson)
7 أكتوبر 2024، 6:24ص
7
نعم. سيختفي في المرة التالية التي تقوم فيها بإعادة تحميل الصفحة.
إعجابَين (2)
ted
(Ted Johansson)
7 أكتوبر 2024، 7:18ص
9
كان هناك إصلاح آخر مطلوب لكي يعمل هذا على الاستضافة الخاصة بنا.
يمكنك الآن تجاهل هذه الرسالة. ومع ذلك، إذا لم يتم إصلاح المشكلة الأساسية، فستعود عند إعادة تحميل الصفحة.
هذا سلوك معروف. الميزة الحالية هي التخلص من الإشعارات “العالقة” أو التي تحدث لمرة واحدة. نحن نعمل على توسيع هذه الوظيفة للسماح لك بـ “تأجيل” هذه الإشعارات لفترة زمنية محددة، لذلك لن تظهر مرة أخرى على الفور.
3 إعجابات
ted
(Ted Johansson)
تم إغلاقه في
21 أكتوبر 2024، 12:00ص
10
تم إغلاق هذا الموضوع تلقائيًا بعد 13 يومًا. لم تعد الردود الجديدة مسموح بها.