dhilt / dharmadict

MIT License
1 stars 2 forks source link

Translator can change his password #36

Closed dhilt closed 7 years ago

dhilt commented 7 years ago

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

Нужен отдельный роут, не /translator/:id/edit/password, а скажем /translator/:id/password – на входе должна проверяться роль 'translator'.

Помимо "Нового пароля" и "Подтверждения" должно быть еще одно поле: "Старый пароль". То есть Переводчик не может сбросить пароль без ввода старого.

Кнопка "Сбросить" сбрасывает все три поля в пустую строку.

На сервере появляется новый обработчик.

dyuvzhenko commented 7 years ago

@dhilt Вроде все готово - https://github.com/dhilt/dharmadict/commit/6f76be9a9527e669371f6b393975b793fa6d4f47.

dyuvzhenko commented 7 years ago

@dhilt И отходя от темы этой задачи, думаю стоит немного поменять подход к показу ошибки. Сейчас мы текст, приходящий с сервера, выдаем за id, который по мнению i18n-библиотеки должен лежать в ru/en.json - в итоге при серверных ошибках в консоли вытекает много ошибок по i18n.

dyuvzhenko commented 7 years ago

@dhilt К примеру это могло бы выглядеть следующим образом - https://github.com/dhilt/dharmadict/commit/79537442b9a9d3e04a525d1dc30dbec9720f601d.

dyuvzhenko commented 7 years ago

@dhilt И вдогонку добавил тесты для серверной части этой задачи - https://github.com/dhilt/dharmadict/commit/663dd4b010d7490046a93e34a2a1d8321ad08539.

Может стоит перенести все тесты для backend в папку test/backend?

dyuvzhenko commented 7 years ago

@dhilt Взял на себя наглость и уже перенес - https://github.com/dhilt/dharmadict/commit/fe5c52cbf6791f834e2435cd5740862f5547a5d3.

dhilt commented 7 years ago

@BitDen Очень хорошие тесты и с /test/backend я согласен. только надо либо moch.opts загнать в /test/backend, либо jest.config.json положить в /test.

dyuvzhenko commented 7 years ago

@dhilt Чуть позже постараюсь переложить mocha.opts в /test/backend, с первой попытки что-то не вышло пока что.