search

sedovalx / taxi

6 stars 3 forks source link

Удаление администратора #36

Open kirzas opened 9 years ago

kirzas commented 9 years ago

Сейчас можно удалить текущего пользователя, т.е. самого себя. Нужно это запретить в виде правила валидации как на сервере, так и на клиенте.

sedovalx commented 9 years ago

Это вопрос? Что конкретно ломается?

kirzas commented 9 years ago

это не вопрос :) просто тут на маке я не знаю, где запятая суть в том, что через интерфейс можно удалить пользователя админ/админ, даже если он единственный пользователь. если его удалить, то залогиниться нельзя. @sedovalx

sedovalx commented 9 years ago

Понял. Это баг, но пока некритичный. Зависит от #19 и #34.

kirzas commented 9 years ago

@sedovalx я правильно сделал, что создал issue?

sedovalx commented 9 years ago

@Argelein да, конечно - все баги нужно фиксировать в виде issue с меткой Bug. Только вот описывать их нужно более конкретно:

sedovalx commented 9 years ago

Сервер теперь запрещает удалять самого себя, однако на клиенте запись в списке все равно пропадает и появляется только при следующей перезагрузке списка. Нужно либо как-то восстанавливать запись при ошибке, либо запрещать на клиенте удалять самого себя.