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

feat: improved user acceptance flow #59

Merged
merged 2 commits into from
Jun 10, 2024
Merged

feat: improved user acceptance flow #59

merged 2 commits into from
Jun 10, 2024

Conversation

hrvadl
Copy link
Collaborator

@hrvadl hrvadl commented Jun 9, 2024

Closes #57

Тепер з ботом можна не вітатися!!!

image

image

image

image

image

@hrvadl hrvadl requested review from vpakhuchyi, mymmrac and andrsj June 9, 2024 12:32
@hrvadl hrvadl self-assigned this Jun 9, 2024
@hrvadl hrvadl added the 🔥 Enhancement New feature or request label Jun 9, 2024
@hrvadl
Copy link
Collaborator Author

hrvadl commented Jun 9, 2024

@mymmrac @vpakhuchyi
Хочу почути вашу думку щодо мого рішення. Не можу зрозуміти чи це костиль чи ні. Tricky part тут у тому, що щоб змінити останнє повідомлення у callback query хендлері ми повинні знати ID останнього повідомлення. Без БД це зробити важкувато.

Я зробив наступне - спочатку відправляю повідомлення з правилами без кнопок accept/decline. Після відправлення отримую ID месаджа, який я кладу у callback query data, щоб у майбутньому використати його. Додаю до цього ж повідомлення кнопки моментально (користувач скоріш за все не встигне й побачити повідомлення без кнопок).

Таким чином, у callback query нам вже прилітає payload з ID останнього повідомлення і замінити його не проблема)

@hrvadl hrvadl force-pushed the feat/accept-improvement branch from 7ca6bee to d6451ca Compare June 9, 2024 13:00
@hrvadl hrvadl force-pushed the feat/accept-improvement branch from 9ea7f42 to 7be6ab3 Compare June 10, 2024 16:53
@hrvadl hrvadl force-pushed the feat/accept-improvement branch from 7be6ab3 to 04fa060 Compare June 10, 2024 16:54
@hrvadl hrvadl merged commit f0b6faa into main Jun 10, 2024
3 checks passed
@hrvadl hrvadl deleted the feat/accept-improvement branch June 10, 2024 18:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🔥 Enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[TB-20]: Add join request successful approval log
2 participants