feat: Implement group pages feature with management and access controls with markdown support - #1721
feat: Implement group pages feature with management and access controls with markdown support#1721fayzetwin1 wants to merge 16 commits into
Conversation
…ls with markdown support
|
|
блять playwright объебался герычем |
|
ща |
добавил таблицы еще |
|
Имхо одной из проблем вк было наличие слишком большого количества сущностей, вот сущностей для длинных текстов как минимум три было. |
|
сделал; 
|
|
Я так понимаю объединение с заметками сделано, но тогда модель викистраницы нужно удалить и вообще полностью объединить с заметками. Завтра гляну на локалхосте |
|
Вообще норм, но пару замечаний душных (можно не реализовывать кроме первого и пятого):
|
|
Имел в виду паддинг от заголовка, но так сойдёт Здесь нужен внутренний отступ для низа страницы по горизонтали на уровне вкладок Оповещение, что нет названия - вместо alert лучше fastError использовать Больше не вижу к чему придраться |
crythoughts
left a comment
crythoughts
left a comment
There was a problem hiding this comment.
а ещё:
поведение кнопки "вставить фото" верни как было
и ещё миграцию 66 нужно объединить с миграцией 65
| "page_tab_view" = "View"; | ||
| "page_tab_edit" = "Edit"; | ||
| "page_tab_history" = "History"; | ||
| "page_preview" = "Preview"; |
There was a problem hiding this comment.
многое из "новых" строк уже есть в локали, переиспользуй старые
| await expect(page.locator('.page_body')).toHaveScreenshot('note-create.png', { | ||
| maxDiffPixels: 200, | ||
| mask: [page.locator('.monaco-editor .scrollbar')], | ||
| }); |
There was a problem hiding this comment.
а зачем в алгоритмы тесты лезть было? то, что динамически меняется, на скриншотах должно быть замазано фиолетовым
| use openvk\Web\Models\Entities\{Note, NoteRevision, User, Club}; | ||
| use Nette\Database\Table\ActiveRow; | ||
| use Nette\Database\Table\Selection; |
There was a problem hiding this comment.
прослеживается очень интересная логика: Entities мы объединим в квадратные скобочки, но вот на Table мы забьём огроменный жирный хуй
| return sizeof($this->table()->where([ | ||
| "owner" => -$club->getId(), | ||
| "deleted" => 0, | ||
| ])); |
There was a problem hiding this comment.
sizeof менять на ->count('*') ВЕЗДЕ. пусть количество подсчитывает mysql сервер, а не php
4 participants
Changelog:
Co-authored by @myslivets <3