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.

Sign up for GitHub

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

Jump to bottom

v2.7 #12

Merged
merged 56 commits into from
May 17, 2021
Merged

v2.7 #12

merged 56 commits into from
May 17, 2021

Conversation

allerter
Copy link
Owner

  • Refactored handlers to make traversing main menu and subsequent menus error-prone.
  • Fixed various bugs
  • Bumped the runtime Python on Heroku to 3.8.10

allerter added 30 commits May 7, 2021 11:45
@allerter
catch badrequest when sending main menu as message edit
9213aca
@allerter
fix issue where track has no number in album
c048536
@allerter
catch badreqeust when getting album lyrics
2372208
@allerter
fix naming conflict
9c0ecd7
@allerter
change main menu regex to ^$
5c063ff 
only enter main menu for /start commands with no leading characters
@allerter
send album progress message as a new message
d65c360 
since users enter get_album from a clicking on a button on a captioned photo, you can't edit that message (maybe you can, but for now let's let it be)
@allerter
separate user feedback from user data in send_feedback
c5bb3fc
@allerter
fixed "wrong http url" error because of inline keyboard
4eb808a 
instead of assigning the strings as callback data, I had assigned them as urls for the inline buttons and that's what the error means: wrong http url
@allerter
get user before welcome_to_shuffle
7433607
@allerter
Revert "change main menu regex to ^$"
8db25d4 
This reverts commit 5c063ff.
@allerter
fix getting token from genius code
ad2aef5
@allerter
separate user feedback from user data when user has a username
c6920a7
@allerter
include bot lang when sending user feedback
9d96202
@allerter
get user before display_album_formats
5312309
@allerter
handle pending updates when starting polling
294f4e0
@allerter
end conversation if no level in user data
49273a7
@allerter
delete start menu after entering welcome_to_shuffle
28cb573
@allerter
accept /start in Shuffle conversation handler
339c757
@allerter
handle all types of updates in Shuffle fallback
d9390ea
@allerter
Add newline at the end of YAML files
ecc73f4
@allerter
add missing import for TypeHandler
c8f7d31
@allerter
Inform user of conversation end when user finishes conversation
625c05d
@allerter
remove start from Shuffle fallbacks
6c02aac
@allerter
log level value in end_describing
4a201ff
@allerter
remove level after user enters main menu
89284e8
@allerter
replace level mechanism with using callback query data
5817563
@allerter
add back button to bot_language
bea5aa2
@allerter
blackified
c69f77f
@allerter
added back button to welcome_to_shuffle
0c78db4
@allerter
fixed back button in welcome_to_shuffle
bab2d57
allerter and others added 26 commits May 10, 2021 00:01
@allerter
delete original message instead of logging exception in main_menu
193965f
@allerter
catch all updates in my_states fallbacks
61d5b56
@allerter
use callback query data in end_describing
ad8a2f3
@allerter
fix missing parameter for TypeHandler
458b325
@allerter
fix misplaced callback_data in lyrics_language
8132b25
@allerter
blackified
4f26b8f
@allerter
update states patterns
7955ef6
@allerter
inform user about /cancel in type_preferences
f8d4e9e
@allerter
add commands to handler that don't need input outside conversation
5d2412b
@allerter
bring lyircs_language, bot_lang and include_annotation to top level
fb19646 
don't use states to handle user choosing an option. instead bring them up to the top level and assign unique values to their callback data
@allerter
update constants
6c3f617
@allerter
update callback queries
5d3b3b7
@allerter
return END instead of conversationhandler state in account
192a715
@allerter
blackified
084e00e
@allerter
fix include annotations pattern
f5d7f58
@allerter
return END in main_menu
9d0eed0
@allerter
add main menu handler separately
30d0d5c
@allerter
add callback query handler to the commands conversation handler
3fee395
@allerter
F541: remove f from string with not placeholder
ca67528
@allerter
updated help message
61e654d
Bump Heroku runtime Python to 3.8.10
e8bc37b
re-enable error handler
a08dc9d
Improve bot.error
51ce1df 
Added more info for user and logs
Fix HTTPError has no attribute status_code
bb42bc1
remove redundant return statement in bot.end_describing
4e100f4
Tests: updated tests
2bd4d6f
@allerter allerter merged commit 7781350 into master May 17, 2021
@allerter allerter deleted the develop branch May 17, 2021 05:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@allerter