kamabzalov / admss-client

Adms react app
0 stars 0 forks source link

Changed inventory group default options #478

Closed JSDID closed 3 months ago

JSDID commented 3 months ago

photo_2024-07-09_14-44-55 (2) photo_2024-07-09_14-44-55

Если создавать группу с тем именем что у нас уже есть, нам сервер возвращает просто ошибку 422 Invalid parameters (т.е. не указывается, что такая группа уже существует). Отсюда вопрос: Нам просто на фронте прописать ошибку общую, например: "Error while create inventory group", или же все таки будет ответ от сервера более конкретный?

JSDID commented 3 months ago

photo_2024-07-09_14-50-46

По поводу order: Тут проблема в том, что order присваивает бек. И при создании новой группы, ей присваивается order 0, даже если группа с order 0 уже есть. И в этом и проблема, что у нас группы могут быть с одинаковым order, а так же есть проблема с пропусками. Допустим у нас есть 3 группы, у которых на беке может быть order 0 0 2. Соответствено, если мы помытаемся переместить первую или вторую группу ниже - у нас ничего не отработает, потому что order у нас станет 0 1 2. Поэтому было бы хорошо, что-бы на беке убирались пропуска и дубли order

JSDID commented 3 months ago

photo_2024-07-09_20-40-03

Нам нужно выйти из функции, показать тост и при этом завершить загрузку. Если этого не сделать у нас при ошибке просто будет постоянный loader