search

sedovalx / taxi

6 stars 3 forks source link

Клиент отправляет неверное время операций на сервер #100

Closed sedovalx closed 9 years ago

sedovalx commented 9 years ago

При создании операции поле changeTime хранит в себе только дату, но не время операции. Т.е. на сервер всегда уходит полночь. Из-за этого сбои в расчетах. Нужно либо отказать от ручного выбора времени при создании операции (вовсе убрать это поле из UI), либо заменить компонент для выбора времени. Например, на такой https://eonasdan.github.io/bootstrap-datetimepicker/.

В случае отказа нужно при сохранении операции руками проставлять в это поле текущее клиентское время.

@hodkoff2 отказаться - это быстрый и вполне себе рабочий вариант. Конечно легко представить ситуацию, когда платеж нужно завести "на вчера", но на первое время и так должно пойти. Как ты считаешь?

Или, если @Argelein чувствует в себе силы разобраться с использованием предложенного мной контрола, то вариант замены представляется более привлекательным.

sedovalx commented 9 years ago

Юбилейная задача, кстати :birthday: :cocktail: :collision: :tada:

sedovalx commented 9 years ago

Пока сам попробую, тут все неочевидно.

sedovalx commented 9 years ago

@Argelein теперь для выбора времени используй контрол как в мастере операций. Его описание есть в шапке задачи. Можно использовать его в режиме выбора только даты или только времени.

hodkovdd commented 9 years ago

@sedovalx Все еще нет операций в истории аренды. Стенд обновил сегодня. 38

... Нужно либо отказать от ручного выбора времени при создании операции (вовсе убрать это поле из UI) ... Убираем.

kirzas commented 9 years ago

https://github.com/sedovalx/taxi/issues/94#issuecomment-123834421 у меня работает

hodkovdd commented 9 years ago

Ок, проверил на стенде, закрываю.