search

kamabzalov / admss-client

Adms react app
0 stars 0 forks source link

Inventory - доработки #404

Closed kamabzalov closed 5 months ago

kamabzalov commented 5 months ago

При сохранении инвентории не приходит никакого сообщения, что она сохранилась, а также не возвращает на лист всей инвентории. В результате чего, нет ощущения, что инвентория сохранилась Также надо добавить валидацию (как уже сделано в deals)

image

Шаги:

  1. Зайти в приложение
  2. Перейти в создание инвентории
  3. Заполнить обязательные поля
  4. Сохранить инвенторию AR: нет оповещения о удачной операции, нет возврата на лист инвентории ER: сообщение о удачности совершеня операции и переход в лист инвентории
kamabzalov commented 5 months ago

Также изменения, которые требуется реализовать:

Поля location, inventory group, stock# сделать required Убрать точку в поле mileage, оставить 1 ноль, сделать НЕобязательным Сделать, чтобы чекбоксы в разделе audit всегда отображались внезависимости от маштаба, сейчас то они есть, то их нет Убрать хедер с моделью, маркой и т.д. в карточке создания, оставить только в карточке редактирования В карточки инвентории в Export to WEB -> Price and comments все поля можно редактировать и без галочки в Export to web, также поле List price НЕ должно быть обязательным. При заполнении полей, где открываются модальные окна со списками, кнопка Add new кидает на 404, а не на форму создания Добавить валидацию на создание инвентории, если какие-то из обязательных полей не заполнены

kamabzalov commented 5 months ago

Здесь же попробовать реализовать подцветку (кружок) невалидной вкладки

kamabzalov commented 5 months ago

@JSDID обязательны только те поля, которые в задаче

kamabzalov commented 5 months ago

@JSDID

  1. Некоторые обязательные поля не подсвечиваются красным, при отработки валидации, а под полем Inventory group и ошибка не показывается image image
  2. Все еще ловится ситуация, когда AUDIT пустой image
  3. После сохранение, показывает сообщение - Deal saved successfully
  4. Сохраняется без обязательного поля типа инвентории, и которое в PURCHASES
  5. Хотя я выбираю тип топлива, это поле не сохраняется ни при создании инвентории, ни при ее редактировании
  6. При заполнении полей General сбрасывается дроп Inventory group, если до этого он был выбран
  7. Нельзя заполнить дроп Body Type, если я выбираю что-то, дроп все равно остается пустым. То же касается дропов Drive Line, Cylinders, Engine description
  8. Поcле первой сохраненной инвентории, последующие вроде как сохраняются, приходит 200, но я их не вижу потом в листе, а вот по айдишке заходит. Пример инвентория с айди f55bbb51-d3ff-4776-91ae-c12122c1a55f под персоной mobile1 password
  9. После попытки изменить инвенторию в айди выше получаю - GroupClass must be a number type, but the final value was: NaN (cast from the value "b9b42938-13a3-4690-9f38-2e69e99947b7"). Выглядит ошибка не очень) image
  10. Отмеченные Options не сохраняются
  11. Если у нас заполнены все поля, по когда мы находится в медии, кнопка сохранения инвентории серая
  12. После ввода поля в разлеле Price and comments, название поля не ушло наверх image
kamabzalov commented 5 months ago

@JSDID при создании инвентори в поле "GroupClass" передаём ИМЯ а не UID

kamabzalov commented 5 months ago

@JSDID новый апдейт:

  1. Type of Fuel не подсвечивается и ошибка не выводится при валидации image
  2. При выборе Purchased From поле остается пустым. Но можно самому вписать что-то, например 123, тогда дает сохранить, но после сохранения поле остается пустым
  3. При сохранении Purchase Expenses - дает 403, но на фронте никак не отображает этого
  4. Кнопка save в медиа всегда серая
  5. После перехода в медию все заполненные данные в инвентории стираются image
  6. Export to WEB Dates нельзя выбрать каледнарь
  7. Все остальные подменю в Export to WEB тоже нельзя редактировать
  8. Установить эти поля обяательными General - Location, inventory group, vin, stock, make, model Description - type of fuel Purchases - purchased from
  9. stocknumber - еси не задано, бэк сам сгенерит этот номер можно у бэка запросить GET /api/v1/inventory/stocknumber GET /api/v1/inventory/{VINcode}/stocknumber
kamabzalov commented 5 months ago

@JSDID новый апдейт:

  1. При удалении медии, если нет прав дает 403, но на фронте эта ошибка никак не отображается
  2. Если зайти с юзера mobile, mobile2 и перейти в лист инвентории. то появляется бесконечная загрузка (пароль password)
  3. На пользователи mobile2, password, если перейти в создание инвентории, то уже будет как будто выбрана локация. Если выбрать ее повторно, то система все равно показывает будто дроп пустой image image
  4. Type of Fuel все еще не подсвечивает image
  5. При выборе компании ничего не происходит
  6. При создании Трат, возвращает 422, но на фронте никак не показывает, что получена ошибка
  7. При создании платежа, он создается получаем 200, но в листе платежей он не появляется
  8. Export to WEB -> Extra data нельзя изменять поля
  9. В медии все еще кнопка сохранения серая
  10. При удалении инвентории показывает уведомление, что все гуд, хотя бек вернул 403 image
kamabzalov commented 5 months ago

@JSDID апдейт (11 июня)

  1. Если у нас стоит какая-то локация в листе инвентории, то она должна автоматически прописываться и в карточке создания инвентории. Все равно это поле обязательное и быть пустым не может. Сейчас она прописывается странно image
  2. После перехода на след пункт, по нажатию на кнопку NEXT, кнопка продолжает подсвечивать зеленым цветом image
  3. По дизайну в компонентам с долларами, всегда 0.00, если нет данных, а по факту пусто image
  4. В print forms кнопки print, download, clear all должны быть вне скролла
  5. Кнопка clear all должна быть серая, если ничего не выбрано
  6. При удалении медии, если нет прав дает 403, но на фронте эта ошибка никак не отображается
  7. Если зайти с юзера mobile, mobile и перейти в лист инвентории. то появляется бесконечная загрузка (пароль password)
  8. На пользователи mobile2, password, если перейти в создание инвентории, то уже будет как будто выбрана локация. Если выбрать ее повторно, то система все равно показывает будто дроп пустой
  9. При выборе компании ничего не происходит
  10. При создании Трат и платежа, возвращает 422, но на фронте никак не показывает, что получена ошибка
  11. При создании платежа, он создается получаем 200, но в листе платежей он не появляется
  12. Export to WEB -> Extra data нельзя изменять поля
  13. В медии все еще кнопка сохранения инвентории серая
  14. При удалении инвентории показывает уведомление, что все гуд, хотя бек вернул 403
  15. При создании инвентории выбираю тип топлива, но фронт говорит, что поле невалидно. Также при создании контакта из инвентории меня перекидывает на форму контакта, но данные по инвентории теряются. Надо либо сохранять введенные даныые в store, либо открывать контакты в новом окне - https://monosnap.com/file/a4IlTffS3XlYoh70rZMmBpXV6g79TH
kamabzalov commented 5 months ago

@JSDID update (13.06)

  1. Если у нас стоит какая-то локация в листе инвентории, то она должна автоматически прописываться и в карточке создания инвентории. Все равно это поле обязательное и быть пустым не может.
  2. сохраняет инвенторию, даже если поле locations пустое
kamabzalov commented 5 months ago

@JSDID

  1. Если зайти в уже созданную ивенторию, и перейти в Purchase → Expenses/Payments и нажать на кнопку сохранение платежа или траты, но карточка закрывается и появляется сообщение, что ивентория успешно сохранена.
  2. Для уделания инвентории, причина удаления должна быть обязательной. Сейчас можно удалить не указывая причину
  3. Export to WEB -> Extra data нельзя изменять поля