dhilt / dharmadict

MIT License
1 stars 2 forks source link

New term API: sanskrit #20

Closed dhilt closed 7 years ago

dhilt commented 7 years ago

4 дополнительных поля должны появиться у нового термина. На примере термина "chos nyid" –

sanskrit_eng: "dharmata" sanskrit_eng_lower: "dharmata" sanskrit_rus: "дхармата" sanskrit_rus_lower: "дхармата"

Для этого нужно добавить контролов на http://dharmadict.ru/newTerm, по одному инпуту на каждый язык из common.languages. В данном случае это будут sanskrit_eng и sanskrit_rus. Научиться сохранять их на сервере; _lower версии получаются через .toLowerCase().

Дописать тесты на создание термина!

dyuvzhenko commented 7 years ago

@dhilt Вроде готово - https://github.com/dhilt/dharmadict/commit/cd482386a564d07cb7eca08d9b751c31e2fbcd8f. После создания термина, нам в ответ приходит лишь id термина, так что совсем чуточку в тестах дописал. Валидацию данных на клиенте пока не придумал (навешивание disabled на кнопку). Заранее неизвестно сколько будет полей в объекте state.admin.newTerm.sanskrit, и наверное надо делать какую-то мудреную переменную, которая будет собирать в себе результат прохода по массиву state.common.languages и сравнивании с state.admin.newTerm.sanskrit.

dhilt commented 7 years ago

@BitDen Я провел рефакторинг кода, касающегося этой части. Осталось довести тесты, это я предлагаю сделать тебе:

Внутри then необязательно писать return Promise.resolve(some); достаточно просто return some; – then сам обернет это в resolved промис. Я переписал контроллер terms в соотв. с этим.

Если по моим комитам есть вопросы -- пиши!

dyuvzhenko commented 7 years ago

@dhilt С коммитами все вроде как понятно. Тесты дописал - https://github.com/dhilt/dharmadict/commit/9cde954620f028e5f3f7211510719ccb0567738a.