Skip to content

fix(messages): implement message attachments#1514

Open
fgRuslan wants to merge 21 commits intoOpenVK:masterfrom
fgRuslan:message-attachments
Open

fix(messages): implement message attachments#1514
fgRuslan wants to merge 21 commits intoOpenVK:masterfrom
fgRuslan:message-attachments

Conversation

@fgRuslan
Copy link
Copy Markdown
Contributor

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

Fix #212

@fgRuslan
Copy link
Copy Markdown
Contributor Author

сука линтер, ща исправлю

@veselcraft
Copy link
Copy Markdown
Member

image

@fgRuslan
Copy link
Copy Markdown
Contributor Author

нада в один коммит мержить или нинада?

@veselcraft
Copy link
Copy Markdown
Member

у нас и так squash всегда используется при слиянии

@koke228666
Copy link
Copy Markdown
Contributor

герой
осталось сделать диалоги

@fgRuslan
Copy link
Copy Markdown
Contributor Author

герой осталось сделать диалоги

допустим что я это сделаю, а что имеется в виду? как эти диалоги должны выглядеть?

@xRy0
Copy link
Copy Markdown
Contributor

xRy0 commented Apr 22, 2026

герой осталось сделать диалоги

допустим что я это сделаю, а что имеется в виду? как эти диалоги должны выглядеть?

Ну чаты, чтобы можно было создавать чат на несколько пользователей, как группы в телеграмме

@fgRuslan
Copy link
Copy Markdown
Contributor Author

нифига себе конечно

@fgRuslan
Copy link
Copy Markdown
Contributor Author

хотел как лучше, а получилось как всегда, извиняюсь за кривой rebase, на данный момент добавил трейт TMessageAttachment для фото и видео, добавил также поля в таблицы videos и photos, которые обозначают, принадлежат ли эти фото или видео какой-то личке. Если да, то перед их отображением происходит проверка, является ли пользователь, который хочет посмотреть это фото/видео, участником диалога, где это фото/видео было выложено. Если да, то отображается нормально, если нет то 403.

@WerySkok
Copy link
Copy Markdown
Member

Ты понимаешь, что с такой историей коммитов ревьюить твой ПР невозможно?

У тебя вместо rebase ПОВЕРХ мастер-ветки (с последующим git push --force) или merge из мастера, которые обрабатываются гитхабом нормально, видны ребейсы ИЗ мастера. Итог — во вкладке «Files Changed» отображаются изменения из других веток, которые здесь не нужны, что делает кодревью невозможным.

https://git-scm.com/book/en/v2/Git-Tools-Rewriting-History

@fgRuslan
Copy link
Copy Markdown
Contributor Author

ладна прасти я скоро исправлю

@fgRuslan fgRuslan force-pushed the message-attachments branch from ac033d1 to 1c8ae51 Compare April 25, 2026 20:15
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.

Прикрепить в сообщениях

5 participants