Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Closed
kamabzalov opened this issue Jun 17, 2024 · 11 comments
Closed
Assignees

Comments

@kamabzalov
Copy link
Owner

kamabzalov commented Jun 17, 2024

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

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

JSDID commented Jun 18, 2024

Вопрос, как #430 в этой задаче может быть "Если при создании не указан Stock#" если он required

photo_2024-06-18_09-23-17

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

@kamabzalov
Copy link
Owner Author

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

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

@kamabzalov
Copy link
Owner Author

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

@kamabzalov kamabzalov changed the title Автоматическое создание сток номера Поле stock number. Проверка на уникальность Jun 19, 2024
@kamabzalov
Copy link
Owner Author

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

image

@kamabzalov
Copy link
Owner Author

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

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

@kamabzalov
Copy link
Owner Author

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

@kamabzalov
Copy link
Owner Author

@JSDID update (25.06.2024)

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

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

А не такой

image

@kamabzalov
Copy link
Owner Author

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

@JSDID
Copy link
Collaborator

JSDID commented Jul 2, 2024

@kamabzalov
Copy link
Owner Author

kamabzalov commented Jul 2, 2024

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

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

@kamabzalov
Copy link
Owner Author

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

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants