-
Notifications
You must be signed in to change notification settings - Fork 52
Telegram v2 #328
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
Merged
Merged
Telegram v2 #328
Changes from all commits
Commits
Show all changes
41 commits
Select commit
Hold shift + click to select a range
12cc75c
Minor changes
leddcode f5f29c0
Merge branch 'develop' of https://github.com/PythonFreeCourse/calenda…
leddcode a7793c3
Add telegram client
leddcode bfe2ceb
Apply merge changes
leddcode 7af4147
Minor fixes
leddcode 8433c4a
Minor fixes
leddcode 091db8e
Create FastAPI Settings object
leddcode 8e93a36
Use status code names in telegram tests
leddcode b8d0cba
Use status code names in telegram tests
leddcode 198e4d6
Add Asynchronous Testing and Increase Code Coverage.
leddcode 3fc9d49
Added keyboards, handlers and tests. Updated telegram models.
leddcode 798a72f
Update tests.
leddcode 3078682
Minor fixes.
leddcode 94a8c01
Add test bot client with registered user
leddcode b0bc592
Minor fix
leddcode de2995c
Make telegram bot working async. Add async fixture and tests.
leddcode e618089
Resolve conflict on merge
leddcode 2122222
Add telegram tests
leddcode c1d309a
Make request.post works asynchronous
leddcode 8795a7b
Merge updates
leddcode a5b4a6f
Minor change
leddcode 08e65de
Merge updates to telegram
leddcode 6229fc0
Add an ability to create events on telegram
leddcode f86f5a1
Merge branch 'develop' of https://github.com/PythonFreeCourse/calenda…
leddcode 245c0c0
Merge updates from develop
leddcode 316c3ab
Test creating new events
leddcode 5dcc8ac
Minor fix
leddcode 2bc356a
Merge changes
leddcode 68d75d2
Splitting to functions
leddcode a9d8205
Merge updates
leddcode 28fe8e8
Merge branch 'develop' of https://github.com/PythonFreeCourse/calenda…
leddcode 2ff5de3
Minor changes
leddcode 2a20efc
Merge changes
leddcode 2086e56
Set and drop webhook asynchronously
leddcode 73cb88c
Merge requirements.txt
leddcode 1137770
Merge requirements.txt
leddcode 498ef67
Minor fix
leddcode 68e5c92
Merge
leddcode 87e069b
Merge branch 'develop' of https://github.com/PythonFreeCourse/calenda…
leddcode ad43971
Change telegram bot url
leddcode 5b6b73b
Minor fix
leddcode File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Binary file not shown.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -148,4 +148,4 @@ win32-setctime==1.0.3 | |
word-forms==2.1.0 | ||
wsproto==1.0.0 | ||
yapf==0.30.0 | ||
zipp==3.4.0 | ||
zipp==3.4.0 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -140,7 +140,7 @@ async def test_user_not_registered(telegram_client): | |
assert response.status_code == status.HTTP_200_OK | ||
assert b'Hello, Moshe!' in response.content | ||
assert b'To use PyLendar Bot you have to register' \ | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I think we shouldn't use \ for line breaks. |
||
in response.content | ||
in response.content | ||
|
||
@staticmethod | ||
@pytest.mark.asyncio | ||
|
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
prefer response.ok