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

Добавление параметров для отправки в telegram #31

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

Conversation

8095tores
Copy link

Параметры для отправки сообщения в telegram описаны https://core.telegram.org/bots/api#sendmessage

Реализовано использование дополнительных параметров. Инициализация дополнительных параметров вынесено в public api библиотеки. Добавлено примитивное описание параметров в коде, но со ссылками.
Реализована поддержка MarkdownV2 (Markdown deprecated в telegram).

КонецЕсли;

// Защищает содержимое отправленного сообщения от пересылки и сохранения
Если ДополнительныеПараметры.Свойство("ЗащищенныйКонтент") Тогда
Copy link
Member

Choose a reason for hiding this comment

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

Неплохо бы документировать, какие вообще "дополнительные параметры" возможны. А лучше сделать не структуру, а специальный класс со всеми свойствами:

ДопСвойства = Новый ПараметрыОтправки();
ДопСвойства.ЗащищенныйКонтент = Истина;

Copy link
Member

Choose a reason for hiding this comment

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

Кстати, внутрь этого класса можно будет инкапсулировать и "вернуть перевод" и сами английские параметры фактической отправки. Получится Json = ПараметрыОтправки.СвойстваСообщения()

Copy link
Author

@8095tores 8095tores Apr 27, 2022

Choose a reason for hiding this comment

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

имеется ввиду сделать отдельный класс для всех Транспортов и инициализировать его на этапе использования библиотеки?
Чтобы было что-то подобное:
Мессенджер = Новый Мессенджер();
Мессенджер.ИнициализироватьТранспорт("telegram", СтруктураАутентификации);
ДопСвойства = Новый ПараметрыОтправки();
ДопСвойства.ЗащищенныйКонтент = Истина;
Мессенджер.ОтправитьСообщение("telegram", ИдентификаторКанала, Сообщение, ДопСвойства);

Copy link
Member

Choose a reason for hiding this comment

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

Я имел в виду, что Структура для пользователя библиотеки это черный ящик. Что в нее можно класть? Какие свойства поймет библиотека? Без залезания в код - не выяснить. Нужна или документация или класс с явными свойствами вместо Структуры.

Copy link
Author

Choose a reason for hiding this comment

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

ок.
попробую перекрутить

@EvilBeaver
Copy link
Member

@8095tores подниму тему?

@@ -73,10 +73,15 @@

КонецЕсли;

// При отпралке сообщений с parse_mode HTML, Markdown или MarkdownV2
// необходимо дополнительно экранировать сообщение

Choose a reason for hiding this comment

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

В PR 34 отпала необходимость экранивания.
Все параметры убраны в тело post-запроса.

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.

3 participants