Feature(bot): inline-button for registrations #294

Hrushon · 2023-03-27T18:34:55Z

Added InlineButton and MenuButton to call registration process template.

https://www.notion.so/Bot-cae1343ec01d42a897219b42f3add7ae

AntonRev

Некоторые сворачивают кнопку и не могут потом ее найти, поэтому нужны еще инлайн кнопка и в меню.
Инлайн кнопка у тебя выдается обрезанная.
В меню можно добавлять без FotFather https://docs.python-telegram-bot.org/en/stable/telegram.menubuttoncommands.html

AntonRev

Мне кажется добавление кнопок везде звучало лучше чем получилось. Может стоит убрать в меню? @Ivan-Skvortsov @ADesBiysk

ADesBiysk · 2023-04-02T19:25:34Z

Мне кажется добавление кнопок везде звучало лучше чем получилось. Может стоит убрать в меню? @Ivan-Skvortsov @ADesBiysk

Мне inline кнопка не очень нравится. Я снял видео и задал вопрос в coreteam.

Ivan-Skvortsov · 2023-04-06T07:22:24Z

Насколько я понял из доков телеги, для вебапп, запущенного из инлайн кнопки и из меню можно использовать answerWebAppQuery .
Разница в том, что sendData - это js-функция из пакета телеги telegram-web-app.js и её можно дернуть на фронте, а answerWebAppQuery - это метод бота, который нужно дергать на бэке.
Получается, нам нужен дополнительный эндпойнт на бэке, на который будет получать данные из вебапп.

Если резюмировать, то порядок действий для получения данных из вебапп должен быть такой:

Получаем данные из initData при помощи кастомного js-скрипта.
При помощи того же скрипта отправляем данные на бэк на наш эндпойнт
На бэке обрабатываем данные, как нам необходимо и вызываем answerWebAppData, чтобы отправить сообщение пользователю (тут нужно использовать web_app_query_id, чтобы не промахнуться с чатом)

P.S.: по сути, даже в answerWebAppQuery нет нужды, мы просто можем повесить на мейн баттон вебаппа скрипт, который выполняет п.п.1 и 2 и закрывает вебапп. А сообщение пользователю отправляем уже с бэка, в зависимости от результатов обработки

P.P.S.: ну и чтобы не делать дополнительные шаблоны и логику на бэке, есть смысл переделать и также и текущую реализацию.

Могу где-то ошибаться, поправьте меня)

Hrushon · 2023-04-08T06:22:18Z

Ребята, подскажите @Ivan-Skvortsov @ADesBiysk @AntonRev
То, что сейчас реализовано, я так понимаю - никого не устраивает?) Нужно переделывать?

Я просто не знаю js и поэтому очень много времени трачу на него :( не хотелось бы зависать над решением, которое не пригодится впоследствии)

ADesBiysk · 2023-04-11T16:43:58Z

Ребята, подскажите @Ivan-Skvortsov @ADesBiysk @AntonRev То, что сейчас реализовано, я так понимаю - никого не устраивает?) Нужно переделывать?

Я просто не знаю js и поэтому очень много времени трачу на него :( не хотелось бы зависать над решением, которое не пригодится впоследствии)

Сейчас реализовано не так как хотелось бы. Пример, как должно работать можно посмотреть в этом боте: https://t.me/DurgerKingBot
Если данная задача требует много времени и усилий, то лучше взять другую и не мучить себя.

Hrushon · 2023-04-12T02:08:17Z

Понял, буду переделывать

Version #2. Added InlineButton & MenuButton to call registration process template. Added new handlers.

…aryery_backend into feature/new_registration_buttons

Version # 2. Added InlineButton & MenuButton to call registration process template. Added new handlers. Temporarily commented out checking of the secret token in the `webhook` endpoint.

Version # 2. Added InlineButton & MenuButton to call registration process template. Added new handler. Temporarily commented out checking of the secret token in the `webhook` endpoint.

Hrushon · 2023-04-16T17:51:08Z

@Ivan-Skvortsov @ADesBiysk @AntonRev

Понятно, что это не окончательный вариант, посмотрите, что выходит. Так пойдет?

Закомментировал проверку секретного ключа в вебхуке телеграм бота на время, чтоб проходили POST-запросы с шаблона регистрации. Пока не знаю - как в шаблоне секретный ключ получить.
Вроде все неплохо, единственное, не очень смотрится, что сообщение метод answerWebAppQuery от имени пользователя отправляет.
Может объединим обратно методы update_user_data и register_user? Столько повторяется там кода...

src/api/routers/telegram_webhook.py

Ivan-Skvortsov · 2023-04-21T12:39:09Z

src/templates/registration/registration.html

+          }
+        });
+      } else {
+        tg.sendData(JSON.stringify(data));


Предлагаю от sendData вообще отказаться и все кейсы (кейбоард кнопка, инлайн кнопка и меню) обрабатывать одинаково.

Если это возможно, конечно)

скорее невозможно, так как данные о чате и пользователе отсутствуют при активации формы регистрации с keyboard button в initDataUnsafe, а больше и взять то негде

Неприятно, конечно)
Получается, нам нужно либо отказываться от кейбоард кнопки, либо писать отдельно логику для кейбоард и отдельно для инлайн/меню?

почему отказываться?
сейчас же работает. чуток совсем добавилось логики для инлайн/меню кнопок.
для пользователя разница лишь в способе оповещения о получении данных с формы: при активации с кейборд кнопки - встроенное телеграмовское оповещение, а при активации с меню/инлайн кнопки - отправка ботом сообщения от имени пользователя в чат.

src/bot/handlers.py

Ivan-Skvortsov · 2023-04-21T13:52:08Z

src/bot/main.py

+    bot_instance.add_handler(
+        TypeHandler(
+            dict,
+            get_web_app_query_data,
+        )
+    )
+    bot_instance.add_handler(MessageHandler(Regex(RE_PATTERN), web_app_data))


Довольно запутанно получилось, на мой взгляд)

Если я правильно понял назначение всех этих хэндлеров - ты пытаешься работать вокруг бота/чата телеги.
Попробуй посмотреть на эту задачу с другой стороны.
Фактически, у тебя есть полноценный фронт (вебапп) и полноценный бэкенд (собственно, наше приложение). При этом в бэкенде у тебя есть доступ к боту (bot_instance, который лежит в app.state).
То есть, для обработки данных из вебапп, чат телеги тебе не нужен вообще. Просто работай с ними как с любыми другими данными)

Вот, например, я накидал тестовый эндпойнт, который принимает запрос с данными из вебапп и отправляет пользователю инлайн сообщение, что все ок (взял кусок из твоего кода).

@Ivan-Skvortsov
если это для того, чтобы только избавиться от функции get_web_app_query_data, то да, спасибо, попробую сделать так.

@Ivan-Skvortsov эта функция будет срабатывать получается вразрез с очередью бота update_queue, не могу оценить - насколько это критично? ощущение, что это не очень хорошо

Мне кажется, в данном случае нам это неважно. Я думаю, работу с вебапп нужно рассматривать как работу бэка с фронтом)
Могу ошибаться, конечно)

Version # 3. Added InlineButton & MenuButton to call registration process template. Added new handler. Added new endpoint for a requests with data coming from registration form launched via the keyboard button.

…aryery_backend into feature/new_registration_buttons

Version # 4. Added InlineButton & MenuButton to call registration process template. Added new handler. Added new endpoint for a requests with data coming from registration form launched via the keyboard button. Changed the function of the registration template rendering in terms of adding a url from the settings-file to receive requests with data launched via `inline/menu` button.

…aryery_backend into feature/new_registration_buttons

src/bot/handlers.py

Version # 5. Added InlineButton & MenuButton to call registration process template. Added new handler. Added new endpoint for a requests with data coming from registration form launched via the keyboard button. Changed the function of the registration template rendering in terms of adding a url from the settings-file to receive requests with data launched via `inline/menu` button.

…aryery_backend into feature/new_registration_buttons

Version # 6. Added InlineButton & MenuButton to call registration process template. Added new handler. Added new endpoint for a requests with data coming from registration form launched via the keyboard button. Changed the function of the registration template rendering in terms of adding a url from the settings-file to receive requests with data launched via `inline/menu` button.

Hrushon temporarily deployed to healthcheck March 27, 2023 18:35 — with GitHub Actions Inactive

Hrushon requested review from ADesBiysk and Ivan-Skvortsov March 27, 2023 18:38

AntonRev reviewed Mar 28, 2023

View reviewed changes

Hrushon temporarily deployed to healthcheck March 29, 2023 17:10 — with GitHub Actions Inactive

Hrushon requested a review from AntonRev March 29, 2023 17:25

AntonRev reviewed Apr 2, 2023

View reviewed changes

Hrushon added 3 commits April 16, 2023 21:52

feat(bot): inline- & menu-button for registrations

7ab145d

Version #2. Added InlineButton & MenuButton to call registration process template. Added new handlers.

Merge branch 'develop' of github.com:Studio-Yandex-Practicum/lomaya_b…

21bad11

…aryery_backend into feature/new_registration_buttons

feat(bot): inline- & menu-button for registrations

e14382d

Version # 2. Added InlineButton & MenuButton to call registration process template. Added new handlers. Temporarily commented out checking of the secret token in the `webhook` endpoint.

Hrushon force-pushed the feature/new_registration_buttons branch from 77e32fa to e14382d Compare April 16, 2023 16:27

Hrushon temporarily deployed to healthcheck April 16, 2023 16:27 — with GitHub Actions Inactive

feat(bot): inline- & menu-button for registrations

70cfff9

Version # 2. Added InlineButton & MenuButton to call registration process template. Added new handler. Temporarily commented out checking of the secret token in the `webhook` endpoint.

Hrushon temporarily deployed to healthcheck April 16, 2023 17:41 — with GitHub Actions Inactive

Hrushon requested a review from AntonRev April 16, 2023 17:51

Ivan-Skvortsov reviewed Apr 21, 2023

View reviewed changes

src/api/routers/telegram_webhook.py Show resolved Hide resolved

Ivan-Skvortsov reviewed Apr 21, 2023

View reviewed changes

src/bot/handlers.py Outdated Show resolved Hide resolved

Ivan-Skvortsov reviewed Apr 21, 2023

View reviewed changes

Hrushon added 2 commits April 21, 2023 22:54

feat(bot): inline- & menu-button for registrations

09368e9

Version # 3. Added InlineButton & MenuButton to call registration process template. Added new handler. Added new endpoint for a requests with data coming from registration form launched via the keyboard button.

Merge branch 'develop' of github.com:Studio-Yandex-Practicum/lomaya_b…

95c776f

…aryery_backend into feature/new_registration_buttons

Hrushon temporarily deployed to healthcheck April 21, 2023 17:01 — with GitHub Actions Inactive

Hrushon and others added 2 commits April 23, 2023 09:43

Merge branch 'develop' into feature/new_registration_buttons

083f627

Hrushon temporarily deployed to healthcheck April 23, 2023 03:58 — with GitHub Actions Inactive

Hrushon requested a review from Ivan-Skvortsov April 23, 2023 03:59

Merge branch 'develop' of github.com:Studio-Yandex-Practicum/lomaya_b…

08e1a5b

…aryery_backend into feature/new_registration_buttons

Ivan-Skvortsov reviewed Apr 25, 2023

View reviewed changes

src/bot/handlers.py Show resolved Hide resolved

Hrushon added 2 commits April 29, 2023 16:29

Merge branch 'develop' of github.com:Studio-Yandex-Practicum/lomaya_b…

b2f54c2

…aryery_backend into feature/new_registration_buttons

Hrushon temporarily deployed to healthcheck April 29, 2023 10:32 — with GitHub Actions Inactive

Hrushon requested a review from Ivan-Skvortsov April 29, 2023 10:33

Hrushon added 2 commits May 21, 2023 15:05

Merge branch 'develop' of github.com:Studio-Yandex-Practicum/lomaya_b…

21db330

…aryery_backend into feature/new_registration_buttons

Hrushon temporarily deployed to healthcheck May 21, 2023 09:46 — with GitHub Actions Inactive

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Feature(bot): inline-button for registrations #294

Feature(bot): inline-button for registrations #294

Hrushon commented Mar 27, 2023 •

edited

Loading

AntonRev left a comment

AntonRev left a comment

ADesBiysk commented Apr 2, 2023

Ivan-Skvortsov commented Apr 6, 2023 •

edited

Loading

Hrushon commented Apr 8, 2023 •

edited

Loading

ADesBiysk commented Apr 11, 2023

Hrushon commented Apr 12, 2023

Hrushon commented Apr 16, 2023 •

edited

Loading

Ivan-Skvortsov Apr 21, 2023

Ivan-Skvortsov Apr 21, 2023

Hrushon Apr 21, 2023

Ivan-Skvortsov Apr 25, 2023

Hrushon Apr 26, 2023

Ivan-Skvortsov Apr 21, 2023 •

edited

Loading

Hrushon Apr 21, 2023 •

edited

Loading

Hrushon Apr 21, 2023 •

edited

Loading

Ivan-Skvortsov Apr 25, 2023

Feature(bot): inline-button for registrations #294

Are you sure you want to change the base?

Feature(bot): inline-button for registrations #294

Conversation

Hrushon commented Mar 27, 2023 • edited Loading

AntonRev left a comment

Choose a reason for hiding this comment

AntonRev left a comment

Choose a reason for hiding this comment

ADesBiysk commented Apr 2, 2023

Ivan-Skvortsov commented Apr 6, 2023 • edited Loading

Hrushon commented Apr 8, 2023 • edited Loading

ADesBiysk commented Apr 11, 2023

Hrushon commented Apr 12, 2023

Hrushon commented Apr 16, 2023 • edited Loading

Ivan-Skvortsov Apr 21, 2023

Choose a reason for hiding this comment

Ivan-Skvortsov Apr 21, 2023

Choose a reason for hiding this comment

Hrushon Apr 21, 2023

Choose a reason for hiding this comment

Ivan-Skvortsov Apr 25, 2023

Choose a reason for hiding this comment

Hrushon Apr 26, 2023

Choose a reason for hiding this comment

Ivan-Skvortsov Apr 21, 2023 • edited Loading

Choose a reason for hiding this comment

Hrushon Apr 21, 2023 • edited Loading

Choose a reason for hiding this comment

Hrushon Apr 21, 2023 • edited Loading

Choose a reason for hiding this comment

Ivan-Skvortsov Apr 25, 2023

Choose a reason for hiding this comment

Hrushon commented Mar 27, 2023 •

edited

Loading

Ivan-Skvortsov commented Apr 6, 2023 •

edited

Loading

Hrushon commented Apr 8, 2023 •

edited

Loading

Hrushon commented Apr 16, 2023 •

edited

Loading

Ivan-Skvortsov Apr 21, 2023 •

edited

Loading

Hrushon Apr 21, 2023 •

edited

Loading

Hrushon Apr 21, 2023 •

edited

Loading