pangbo
28.Декабрь.2025 08:39:02
46
В настоящее время интерфейс registerReviewableActionModal в API плагинов фронтенда регистрирует модальные окна только для старого макета очереди ревью:
Однако новый макет полностью поддерживает эту функцию.
export function registerReviewableActionModal(actionName, modalClass) {
if (Object.keys(defaultActionModalClassMap).includes(actionName)) {
throw new Error(
`Cannot override default action modal class for ${actionName} (mapped to ${defaultActionModalClassMap[actionName].name})!`
);
}
actionModalClassMap[actionName] = modalClass;
}
Меня интересует, было ли это преднамеренным решением по дизайну или просто упущением?
1 лайк
T_Disco:
Модератор может использовать фильтр «Отмечено флагом» в очереди проверки, чтобы работать с элементами, которые они отметили, и поэтому, после того как действие будет выполнено, было бы полезно иметь возможность снять отметку с элемента, чтобы он не отображался в их списке «Отмечено флагом».
Ах, я понял. Разница с новым решением заключается в том, что «снятие отметки» происходит автоматически после выполнения действия по проверке.
В случае если другой модератор отметил элемент, но не выполнил действие, другой модератор может снять отметку, а затем отметить его самостоятельно.
2 лайка
Спасибо, что заметили эту проблему. Конечно, это было упущение.
У меня есть PR с исправлением этой проблемы
main ← reviewable-refresh-api2
merged 03:05AM - 12 Jan 26 UTC
`addPluginReviewableParam` and `registerReviewableActionModal` api calls was not… working for refreshed review UX.
This problem was solved by removing duplicate definitions from reviewable-item component by importing from the reviewable-refresh/item component instead and updating plugin-api imports.
2 лайка
