-
-
Notifications
You must be signed in to change notification settings - Fork 101
Groups: Wall: add suggestions #935
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
Так ну походу тут реально овк за нас теперь пишут, таким образом до релиза дойдем
Отправлено с iPhone
… 31 июля 2023 г., в 10:27, lalka2018 ***@***.***> написал(а):
Данный пр добавляет предложку в группы. Её можно включить, задав "Стену"
Как и в вк, можно поменять текст поста и включить/выключить отображение автора
После принятия поста автору придёт уведомление
You can view, comment on, or merge this pull request online at:
#935
Commit Summary
10729d8 Wall: add early suggestions
eff2b2b Fix br
213f1e2 Fix empty posts
File Changes (19 files)
M ServiceAPI/Wall.php (60)
M VKAPI/Handlers/Wall.php (52)
M Web/Models/Entities/Club.php (17)
A Web/Models/Entities/Notifications/PostAcceptedNotification.php (13)
M Web/Models/Entities/Post.php (5)
M Web/Models/Repositories/Posts.php (63)
M Web/Presenters/GroupPresenter.php (83)
M Web/Presenters/WallPresenter.php (16)
M Web/Presenters/templates/Group/Edit.xml (7)
A Web/Presenters/templates/Group/Suggested.xml (26)
M Web/Presenters/templates/Group/View.xml (12)
A Web/Presenters/templates/components/notifications/6/_14_5_.xml (7)
M Web/Presenters/templates/components/post/microblogpost.xml (8)
M Web/routes.yml (4)
M Web/static/css/main.css (18)
M Web/static/js/al_wall.js (89)
A install/sqls/00039-suggest-posts.sql (1)
M locales/en.strings (41)
M locales/ru.strings (42)
Patch Links:
https://github.com/openvk/openvk/pull/935.patch
https://github.com/openvk/openvk/pull/935.diff
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.
|
|
This comment was marked as off-topic.
This comment was marked as off-topic.
та нахуй они нужны? я топлю за колобков |
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
@iamdelickkk... Так иди и исправь, создай новый метод на замену |
This comment was marked as off-topic.
This comment was marked as off-topic.
Ну так ты кричишь об этом как в жопу ужаленный. |
Ну так не кипятись сильно. Тут почти ничто и никому не должен, а ты начинаешь спамить свою оффтоп хрень под левым PR, мешая обсуждению именно этой фичи.
@lalka2016 тоже не входит в список организации @OpenVK, а ты именно в его PR начал спамить какую-то хрень |
|
А можно ревью🤣🤣🤣🤣🤣🤣🤣🤣🤣🤣 |
|
крутая фича, только дизайн немного хромает |
Теперь оно удаляет все теги а не только <br>
Методы нестандартные немного
n1rwana
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Через API пользователь может удалить предложенную запись, когда она уже опубликована на стене сообщества
veselcraft
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
впринципе неплохо всё но есть пару вопрсов
veselcraft
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
выглядит неплохо, посмотрим что @celestora скажет
их никогда не было в вк (если не считать эмодзи), и в опенвк не будет. забудьте! |
Давайте рофлить👨💻👨💻👨💻
Add tabs for post view, add signer's object in wall get and add person icon in microblog
Исправлено? |
|
Значит можно мержить? |
|
индекс надеюсь в бд на этот раз есть |
No description provided.