Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
kirzas opened this issue Apr 8, 2015 · 6 comments
Open

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

kirzas opened this issue Apr 8, 2015 · 6 comments

Comments

@kirzas
Copy link
Collaborator

kirzas commented Apr 8, 2015

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

  • Удалил самого себя через UI
  • Пользователь удалился. Залогинится больше нельзя.
  • Не должно быть возможности удалять текущего пользователя.
@sedovalx
Copy link
Owner

sedovalx commented Apr 8, 2015

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

@kirzas
Copy link
Collaborator Author

kirzas commented Apr 8, 2015

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

@sedovalx
Copy link
Owner

sedovalx commented Apr 8, 2015

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

@kirzas
Copy link
Collaborator Author

kirzas commented Apr 8, 2015

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

@sedovalx
Copy link
Owner

sedovalx commented Apr 8, 2015

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

  • Что делал
  • Что получил
  • Что должен был получить

sedovalx added a commit that referenced this issue Aug 13, 2015
…ссе + рефакторинг валидации + #36: предотвращение удаления самого себя
@sedovalx
Copy link
Owner

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants