Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

All api 4.2 and 4.3 changes#1418

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
tsnoam merged 6 commits intomasterfromV12
Aug 23, 2019
Merged

All api 4.2 and 4.3 changes#1418

tsnoam merged 6 commits intomasterfromV12
Aug 23, 2019

Conversation

Eldinnie
Copy link
Member

Tests and coverage check needed

markusressel reacted with hooray emojiNukesor reacted with heart emoji
Eldinnieand others added6 commitsJune 5, 2019 15:00
actually works nowAdd testsFix flake8 issues.Add poll argument to Update.Fix pre-commit configFinalizing 4.2No notify on pin
Add LoginUrl to InlineKeyboardButtonAdd reply_markup to message
@AyraHikari
Copy link

Hello, i test that branch, and starting editing my bot script for future update.
But i got an error:

Traceback (most recent call last):  File "/home/ayra/bot/my/__main__.py", line 535, in process_update    handler.handle_update(update, self)TypeError: handle_update() missing 1 required positional argument: 'check_result'

My script is

for handler in (x for x in self.handlers[group] if x.check_update(update)):                handler.handle_update(update, self)                break

I don't know what check_result is mean, when i'm insert check_result="test", bot is not responding when i send a command /start.

Also there is no guide for that inthis guide.

Thank you

@Eldinnie
Copy link
MemberAuthor

ehm...
Why do you run this? Choosing which handler is/and ashould be done by the dispatcher. It might be helpfull if you shared all your code and the intention of what you;re trying to do.

@AyraHikari
Copy link

Sure,here it is

@EldinnieTelegram GithubBot Revised
Copy link
MemberAuthor

@AyraHikari you rewrote the dispatchersprocess_update method. If you want to do that with the new version, you will have to make sure it works the same asDispatcher.process_update.

@AyraHikari
Copy link

Oh i see, maybe i should rewrite all things since many of that deprecated.
Thank you for your response. Sorry for late reply

@TheHolyLoli
Copy link

why is this not merged?

markusressel and Nukesor reacted with thumbs up emoji

@Nukesor
Copy link

Would love to see this merged, especially since 4.4 is already out there!

Andre0512, Brawl345, and calpt reacted with thumbs up emoji

This was referencedAug 4, 2019
@Bibo-Joshi
Copy link
Member

I noticed, that antelegram.loginurl.rst file is missing for the docs.

@tsnoamtsnoam merged commitac60d05 intomasterAug 23, 2019
@NukesorNukesor mentioned this pull requestAug 23, 2019
6 tasks
@Bibo-JoshiBibo-Joshi mentioned this pull requestApr 8, 2020
18 tasks
@github-actionsgithub-actionsbot locked and limited conversation to collaboratorsAug 20, 2020
Sign up for freeto subscribe to this conversation on GitHub. Already have an account?Sign in.
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.

6 participants
@Eldinnie@AyraHikari@TheHolyLoli@Nukesor@Bibo-Joshi@tsnoam

[8]ページ先頭

©2009-2025 Movatter.jp