Skip to content

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 подниму тему?

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