fix(messages): implement message attachments#1514
fix(messages): implement message attachments#1514fgRuslan wants to merge 21 commits intoOpenVK:masterfrom
Conversation
|
сука линтер, ща исправлю |
|
нада в один коммит мержить или нинада? |
|
у нас и так squash всегда используется при слиянии |
|
герой |
допустим что я это сделаю, а что имеется в виду? как эти диалоги должны выглядеть? |
Ну чаты, чтобы можно было создавать чат на несколько пользователей, как группы в телеграмме |
|
нифига себе конечно |
|
хотел как лучше, а получилось как всегда, извиняюсь за кривой rebase, на данный момент добавил трейт TMessageAttachment для фото и видео, добавил также поля в таблицы videos и photos, которые обозначают, принадлежат ли эти фото или видео какой-то личке. Если да, то перед их отображением происходит проверка, является ли пользователь, который хочет посмотреть это фото/видео, участником диалога, где это фото/видео было выложено. Если да, то отображается нормально, если нет то 403. |
|
Ты понимаешь, что с такой историей коммитов ревьюить твой ПР невозможно? У тебя вместо rebase ПОВЕРХ мастер-ветки (с последующим git push --force) или merge из мастера, которые обрабатываются гитхабом нормально, видны ребейсы ИЗ мастера. Итог — во вкладке «Files Changed» отображаются изменения из других веток, которые здесь не нужны, что делает кодревью невозможным. |
|
ладна прасти я скоро исправлю |
ac033d1 to
1c8ae51
Compare

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