Skip to content
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

Posts: add source param #934

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Posts: add source param #934

wants to merge 4 commits into from

Conversation

mrilyew
Copy link
Contributor

@mrilyew mrilyew commented Jul 29, 2023

Данный пр closes #932 и добавляет "источник" у поста.
имаге
изображение
+добавлены новые методы Wall.checkCopyrightLink(), Wall.pin() и Wall.unpin(), у Wall.post() появился параметр copyright

Copy link
Member

@veselcraft veselcraft left a comment

Choose a reason for hiding this comment

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

-206622442_456239367.mp4

Web/Presenters/templates/components/textArea.xml Outdated Show resolved Hide resolved
@mrilyew mrilyew requested a review from veselcraft July 30, 2023 08:19
VKAPI/Handlers/Wall.php Show resolved Hide resolved
VKAPI/Handlers/Wall.php Show resolved Hide resolved
Web/Models/Entities/Post.php Show resolved Hide resolved
Web/Presenters/WallPresenter.php Show resolved Hide resolved
Web/static/css/main.css Outdated Show resolved Hide resolved
Web/static/js/al_wall.js Outdated Show resolved Hide resolved
install/sqls/00038-posts-source.sql Outdated Show resolved Hide resolved
@NoPlagiarism
Copy link
Contributor

Получается... Готовый PR заглох?

@WerySkok
Copy link
Member

WerySkok commented Oct 5, 2023

изображение
:D

@NoPlagiarism
Copy link
Contributor

Пипяу, он уже успел стать неготовым.

@nbt1248
Copy link

nbt1248 commented Oct 5, 2023

Получается... Готовый PR заглох?

исходя из версии на вепуровке сурсы можно прикреплять к постам вне сообщества (записям в профиле e.g.) и насколько я знаю такого в вк не было и нет
@NoPlagiarism

@NoPlagiarism
Copy link
Contributor

@damirap2020 разве?...

За что ты вообще заставил зайти меня в вк... Тебе было принципиально это делать?
Есть там это. Тьфу блин

изображение

@nbt1248
Copy link

nbt1248 commented Oct 6, 2023

@damirap2020 разве?...

За что ты вообще заставил зайти меня в вк... Тебе было принципиально это делать? Есть там это. Тьфу блин

изображение

Блять
Ладно, я ошибся
Признаю
Невнимательно смотрел
Ждëм фикс и мерж

@NoPlagiarism
Copy link
Contributor

фикс

А фикс чего собсна? PR заглох :<

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.

Возможность указать источник в посте, как у вк
6 participants