-
Notifications
You must be signed in to change notification settings - Fork 16
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
base: master
Are you sure you want to change the base?
Conversation
КонецЕсли; | ||
|
||
// Защищает содержимое отправленного сообщения от пересылки и сохранения | ||
Если ДополнительныеПараметры.Свойство("ЗащищенныйКонтент") Тогда |
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.
Неплохо бы документировать, какие вообще "дополнительные параметры" возможны. А лучше сделать не структуру, а специальный класс со всеми свойствами:
ДопСвойства = Новый ПараметрыОтправки();
ДопСвойства.ЗащищенныйКонтент = Истина;
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.
Кстати, внутрь этого класса можно будет инкапсулировать и "вернуть перевод" и сами английские параметры фактической отправки. Получится Json = ПараметрыОтправки.СвойстваСообщения()
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.
имеется ввиду сделать отдельный класс для всех Транспортов и инициализировать его на этапе использования библиотеки?
Чтобы было что-то подобное:
Мессенджер = Новый Мессенджер();
Мессенджер.ИнициализироватьТранспорт("telegram", СтруктураАутентификации);
ДопСвойства = Новый ПараметрыОтправки();
ДопСвойства.ЗащищенныйКонтент = Истина;
Мессенджер.ОтправитьСообщение("telegram", ИдентификаторКанала, Сообщение, ДопСвойства);
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.
Я имел в виду, что Структура для пользователя библиотеки это черный ящик. Что в нее можно класть? Какие свойства поймет библиотека? Без залезания в код - не выяснить. Нужна или документация или класс с явными свойствами вместо Структуры.
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.
ок.
попробую перекрутить
@8095tores подниму тему? |
@@ -73,10 +73,15 @@ | |||
|
|||
КонецЕсли; | |||
|
|||
// При отпралке сообщений с parse_mode HTML, Markdown или MarkdownV2 | |||
// необходимо дополнительно экранировать сообщение |
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.
В PR 34 отпала необходимость экранивания.
Все параметры убраны в тело post-запроса.
Параметры для отправки сообщения в telegram описаны https://core.telegram.org/bots/api#sendmessage
Реализовано использование дополнительных параметров. Инициализация дополнительных параметров вынесено в public api библиотеки. Добавлено примитивное описание параметров в коде, но со ссылками.
Реализована поддержка MarkdownV2 (Markdown deprecated в telegram).