Skip to content

feat(photos): saved photos - #1720

Open
mrcatman wants to merge 4 commits into
OpenVK:masterfrom
mrcatman:saved-photos
Open

feat(photos): saved photos#1720
mrcatman wants to merge 4 commits into
OpenVK:masterfrom
mrcatman:saved-photos

Conversation

@mrcatman

@mrcatman mrcatman commented Aug 1, 2026

Copy link
Copy Markdown
Contributor

простецкая реализация альбома с сохранёнками. не знаю, что ещё добавить, наверно по дефолту сделать закрытым?

@mrcatman

mrcatman commented Aug 2, 2026

Copy link
Copy Markdown
Contributor Author

немного переделал механизм, чтобы фотки копировались в альбом от имени юзера и не зависели от удаления исходника автором. минус - в таблице дублируется хэш и ссылки на файлы, в будущем можно будет вынести их в отдельную подтаблицу типа photo_uploads

@ayaaop

ayaaop commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

одно и то же фото можно добавить в сохраненки несколько раз

@mrcatman

mrcatman commented Aug 3, 2026

Copy link
Copy Markdown
Contributor Author

одно и то же фото можно добавить в сохраненки несколько раз

по-моему в "оригинале" так же всегда было

@veselcraft

Copy link
Copy Markdown
Member

на будущее - если лень поднимать докер и скриншоты обновлять, можно из артефактов просто вынуть скриншоты и заменить их
image

@veselcraft veselcraft left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ayaaop

ayaaop commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

одно и то же фото можно добавить в сохраненки несколько раз

по-моему в "оригинале" так же всегда было

в вк? нет

@crythoughts

Copy link
Copy Markdown
Collaborator
  • Тут сделано с помощью форм, и нет индикатора что фото было добавлено из-за этого. Во вьювере это привёдет к релоаду. Лучше вынести в js, но не как ивент потому что делегирование ивентов может тормозить, а onclick="save_photo(event)". до того как пришёл ответ давать кнопке класс lagged, потом заменять её на некликабельный текст какой-нибудь не меняя вёрстки и маргинов
image
  • Лучше без роута /photo{num}_{num}/save и сделать метод photos.copy, вызывать через await window.OVKAPI.call("", {}):
image
  • если фото копируется то тогда в таблице photos нужно хранить айди ориг фото. в теории можно будет сделать функцию выхода на оригинал но это хз

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants