search

kamabzalov / admss-client

Adms react app
0 stars 0 forks source link

Added edit to user collections #621

Closed JSDID closed 4 weeks ago

kamabzalov commented 1 month ago

@JSDID почемк разные по высоте?

image

kamabzalov commented 1 month ago

@JSDID ховера нет image

kamabzalov commented 1 month ago

@JSDID Я для кого замечания пишу? Где ховер настранице отчета?

kamabzalov commented 1 month ago

@JSDID поиск так и задуман? image

kamabzalov commented 1 month ago

@JSDID почем edit есть не везде? image

kamabzalov commented 1 month ago

@JSDID Add to collection ничего не делает

JSDID commented 1 month ago

Да, поиск так и задуман

В дизайне есть

Поиск проверяли и мерджили

Edit в дизайне тоже не везде! А только в тех коллекциях, к которым у пользователя есть доступ!

kamabzalov commented 1 month ago

@JSDID поле нельзя отредактировать. Почему?

image

JSDID commented 1 month ago

Документы и эти коллекции возвращают ошибку not permitted. И в дизайне сказано что edit только у созданных пользователем коллекций

А там где нет edit - другой useruid

Add to collection - привязка отчёта к коллекции, мы это обсуждали и просили ещё на прошлой неделе, ответа так и нету

JSDID commented 1 month ago

В сваггере вижу похожее что-то на привязку появилось, тоже могу попробовать (add to collection)

kamabzalov commented 1 month ago

@JSDID где ховер??? https://monosnap.com/file/HPZE9FoH2lyjzsTaJwHyY87ECb56Vy штраф

kamabzalov commented 1 month ago

@JSDID я редактирую Favorites, но создается новая коллекция. Вас это не смущает? https://monosnap.com/file/PyTUb6mB0fABnS7zO6WD4HCqLVAM3u

JSDID commented 1 month ago

curl 'https://app.admss.com/api/v1/reports/d51ffe3a-10d3-497e-a4d2-61894dae2b2e/collection' \ -H 'Accept: application/json, text/plain, /' \ -H 'Accept-Language: en-US,en;q=0.9,ru-UA;q=0.8,ru;q=0.7,uk;q=0.6' \ -H 'Authorization: Bearer 3c307104-da7f-493a-98bc-f60e45ae8f5a' \ -H 'Connection: keep-alive' \ -H 'Content-Type: application/json' \ -H 'Origin: http://localhost:3003' \ -H 'Referer: http://localhost:3003/' \ -H 'Sec-Fetch-Dest: empty' \ -H 'Sec-Fetch-Mode: cors' \ -H 'Sec-Fetch-Site: cross-site' \ -H 'User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/128.0.0.0 Safari/537.36' \ -H 'sec-ch-ua: "Chromium";v="128", "Not;A=Brand";v="24", "Google Chrome";v="128"' \ -H 'sec-ch-ua-mobile: ?0' \ -H 'sec-ch-ua-platform: "Windows"' \ --data-raw '{"name":"Test4","documents":[],"itemuid":"1868b9b5-21f5-45cd-b27d-46b808d3c393"}'

photo_2024-09-19_19-39-56

kamabzalov commented 1 month ago

@JSDID при удалении надо список перечитывать.

kamabzalov commented 1 month ago

@JSDID при загрузке показывается на мгновение крестик https://monosnap.com/file/b5chlnqPKfKxccx9xOMDoHCpBmvZku

JSDID commented 1 month ago

Удаление теперь не возвращает ошибку, но сама коллекция все равно не удаляется) Поправил крестик, добавил обновление списка после удаления.

kamabzalov commented 1 month ago

@JSDID favorites - это коллекция, которая формируется из отчетов с признаком isFavorite. А значит надо убрать возможность ее редаткирования и изменения image

kamabzalov commented 4 weeks ago

@JSDID при добавлении/удалении в favorites не вижу смысла в запросе на все коллекции. Ведь изменилась только favorite

kamabzalov commented 4 weeks ago

@JSDID при переносе из коллекции в коллекцию тоже не вижу смысла в перечитывании favorites

JSDID commented 4 weeks ago

@JSDID я же спросил, все проверили? https://github.com/kamabzalov/admss-client/pull/621#issuecomment-2362969636

До сих пор не починино

Так это сделано

Там нет возможности редактирования

kamabzalov commented 4 weeks ago

@JSDID вот почему это вам незаметно???!!! Я открываю отчет: кнопка Update (ok), а заголовок (Create)? https://monosnap.com/file/94LN3BEXrCwbejMp1h2F1JcobW04Dg

kamabzalov commented 4 weeks ago

@JSDID я же спросил, все проверили? #621 (comment) До сих пор не починино

Так это сделано

Там нет возможности редактирования

я вижу: image

kamabzalov commented 4 weeks ago

@JSDID я же спросил, все проверили? #621 (comment) До сих пор не починино

Так это сделано Там нет возможности редактирования

я вижу: image

Отбой! У меня не все файлы забрал гит

JSDID commented 4 weeks ago

@JSDID вот почему это вам незаметно???!!! Я открываю отчет: кнопка Update (ok), а заголовок (Create)? https://monosnap.com/file/94LN3BEXrCwbejMp1h2F1JcobW04Dg

Потому что так в дизайне

kamabzalov commented 4 weeks ago

@JSDID вот почему это вам незаметно???!!! Я открываю отчет: кнопка Update (ok), а заголовок (Create)? https://monosnap.com/file/94LN3BEXrCwbejMp1h2F1JcobW04Dg

Потому что так в дизайне

На вашем месте я бы спросил у меня ( а я у дизайнера), правильно ли это! Или вы опять только исполняете но сами идеи/замечания не высказываете. Дайте ссылку на дизайн

kamabzalov commented 4 weeks ago

@JSDID после создания отчета надо перенаправлять на страницу с коллекциями