Забыли новые разрешения во вкладке «Безопасность» категории

Продолжение обсуждения из Автоматическое добавление разрешений / приглашение о несохранённых разрешениях:

Я думаю, что нам нужно пойти немного дальше в этом вопросе.

Если у вас есть ожидающее изменение (1) и вы нажимаете «Сохранить», мы должны остановить вас и сообщить, что у вас есть ожидающие изменения для применения, в виде вопроса «Вы уверены?».}

6 лайков

Разве нам вообще нужно нажимать «Сохранить»? Мне кажется, что достаточно нажать кнопку «Добавить»?

3 лайка

Мне нравится функция «Сохранить». Это дополнительный шаг для проверки правильности настроек разрешений. Без неё, если я случайно выберу неверную группу в выпадающем списке и нажму :heavy_plus_sign:, доступ будет предоставлен сразу, без возможности исправить ошибку.

3 лайка

Одна из вещей, которые делают это немного более сложным, заключается в том, что кнопка «Сохранить категорию» в настоящее время применяется ко всему содержимому на всех вкладках/панелях. Предупреждение, вероятно, должно появляться, если вы внесли изменения в ЧТО-ЛИБО в любом разделе и нажали крестик.

6 лайков

Привет, @Osama, можешь взяться за это? Должно быть довольно просто, но дай знать, если возникнут какие-либо проблемы.

4 лайка

PR по этому поводу:

Скриншот:

Кнопка «Сохранить категорию» будет вызывать предупреждение, если пользователь изменит одно из выпадающих меню и не нажмёт кнопку «плюс». Они могут снова нажать «Сохранить», и система проигнорирует ожидающее изменение и сохранит всё равно. Нажатие кнопки «плюс» сбросит триггер, поэтому, если после этого пользователь не изменит выпадающие меню, предупреждение не появится.

Также я изменил цвет кнопки «плюс» на основной, чтобы она была более заметной.

Как вам такое решение?

7 лайков