Детальные групповые права доступа для анонимных и авторизованных пользователей

[martin (Martin Brennan)]

@Moin

@martin

В любом случае я одобрил твой PR для компонента, думаю, всё в порядке

Есть ли причина, по которой это ещё не объединено?

Нет, по какой-то причине, кажется, мой мозг решил, что это не PR в организации Discourse Объединю сразу после завершения проверок CI.

@Moin

Я это сделал. Но у меня всё ещё остаётся впечатление, что это помогает только для форумов, добавляющих компонент после моих изменений. Для тех, кто уже использует его, новые значения по умолчанию не применяются (что обычно хорошо!). Так что я всё ещё вижу проблему, заключающуюся в неожиданном изменении поведения для тех, кто уже использует компонент.

Также, знаешь ли ты, что произойдёт, если администратор настроил параметр с группой, которую я добавлю как запрещённую группу в своём обновлении?

Для таких случаев и других в будущем, думаю, лучшим решением, вероятно, будет написание миграции для каждой темы/компонента отдельно Migrate Discourse theme settings