search

kamabzalov / admss-client

Adms react app
0 stars 0 forks source link

Поле stock number. Проверка на уникальность #430

Closed kamabzalov closed 2 weeks ago

kamabzalov commented 1 month ago

Реализовать надо так:

  1. сделать поле stock необязательным
  2. после вводе в поле значения отправлять запрос на проверку наличия такого номера (/api/v1/inventory/stocknumber). Если он есть, то помечать поле/форму как нвалидную. Если нет, сохранять сток внутри инвентории
JSDID commented 1 month ago

Вопрос, как https://github.com/kamabzalov/admss-client/issues/430 в этой задаче может быть "Если при создании не указан Stock#" если он required

photo_2024-06-18_09-23-17

нам валидация не даст его оставить пустым. Или нужно убирать required, или пересматривать логику отработки

kamabzalov commented 1 month ago

@JSDID ответа от бэка

  1. Убираем required
  2. Если мы ввели сток, то надо отправить запрос на бэк, чтобы понять - уникальный он (сток) или нет.
kamabzalov commented 1 month ago

@JSDID обновил описание задачи

kamabzalov commented 1 month ago

@JSDID Если зайти в созданную инвенторию и менять в ней какую-то инфу, то сразу ругается на уже существующий сток номер

image

kamabzalov commented 1 month ago

@JSDID Если я просто нажму на поле, и не буду его изменять, то мне выдаст ошибку, что такой сток номер уже существует

Screencast_from_2024-06-22_22-10-47.webm

kamabzalov commented 1 month ago

@JSDID убирайте ваш предыдущий "костыль" - при существующей инвентории надо делать поле stocknumber disabled

kamabzalov commented 1 month ago

@JSDID update (25.06.2024)

Если создавать инвенторию новую, но вписать сток номер какой-то другой инвентории, то мы получим ошибку, что сток номер занят.

После этого если изменить сток номер и сохранить инвенторию, то в запросе на создание у нас прописывается айди ивентории, у которой тот сток номер, что мы пытались изначально использовать. В результате чего у нас НЕ создается новая инвентория, а перезаписывается та, что имела сток номер, который мы изначально ввели. image

А не такой

image

kamabzalov commented 1 month ago

@JSDID вдогонку: Вернуть возможность редактирования сток номера (валидировать только при изменении, а еще желательнее, если сток номер изменился от исходного, а не просто событие change вызвалось), при редактировании созданной инвентории. Добавить валидация поля - мин 1 символ, макс 20

JSDID commented 4 weeks ago

https://icecreamapps.com/v/bzvkusr

kamabzalov commented 3 weeks ago

@JSDID ответ от тестировщицы на ваше видео: "Возможно важна последовательность действий, вот видео как это я повторяю, повторить получается несколько раз. Легче проверять на персоне, где мало инвентории, я использую mobile2 password"

Screencast_from_2024-07-02_19-49-44.webm

kamabzalov commented 3 weeks ago

@JSDID я замерджил - посмотрим, что скажет тестировщица Но я по прежнему наблюдаю излшиние запросы на stocknumber. Более того - когда я сохраняю инвенторию - то stocknumber зачем-то тоже вызывается

https://monosnap.com/file/6TXM9FX2YN8J4S4OZjZKVtdiSNfDh3