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

Fix/fix backward 6 7 mix with edit menu #41

Merged
merged 3 commits into from
Feb 5, 2024

Conversation

floks41
Copy link
Collaborator

@floks41 floks41 commented Feb 5, 2024

Найдено неожиданное поведение бота.
Описание проблемы 1:
Вводим команду 6 или 7.
Нажимаем Назад.
Получаем приветствие.
Вводим текст.
Текст уходит админу и пользователь получает сообщение в соответствии с пунктами 6 или 7.
А должно быть приветствие.

Описание проблемы 2:
Вводим команду 6 (запрос обратной связи)
Нажимаем кнопку Назад (получаем приветствие)
Вводим пункт 7
(или вводим пункт 7 с клавиатуры без кнопки назад)
Получаем сообщение по пункту 7.
Вводим Свой вопрос.
Администратору и пользователю уходят сообщения как для обратной связи, т.е. пункту 6.
(аналогичная проблема если поменять пункты 6 и 7 местами).
А должны быть сообщения по вопросу - пункт 7.

Описание проблемы 3:
Вводим команду 6 (запрос обратной связи) или 7.
Вводим edit menu
Вводим текст.
Администратору и пользователю уходят сообщения как для обратной связи, т.е. пункту 6.
А должно быть приветствие.

Описание проблемы 4:
Вводим 6 или 7
Вводим 1, 2,…, 5.
Вводим текст.
Администратору и пользователю уходят сообщения как для обратной связи, т.е. пункту 6.
А должно быть приветствие.

Предложения по исправлению в методе VKBot.__read_menu_handler(...) и self.__check_for_service_event(…), __receive_new_value_handler

@floks41
Copy link
Collaborator Author

floks41 commented Feb 5, 2024

Добавлено описание проблем 3 и 4.

@robky robky merged commit 5abacbe into develop Feb 5, 2024
1 check passed
@robky robky deleted the fix/fix-backward-6-7-mix-with-edit-menu branch February 5, 2024 15:42
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.

4 participants