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

Update for filters and if from_user is None #17

Merged
merged 23 commits into from
Oct 29, 2023

Conversation

Jusidama-Bot
Copy link
Contributor

@Jusidama-Bot Jusidama-Bot commented Jan 24, 2023

Fix if user_id is None and if filters is not async

@Jusidama-Bot Jusidama-Bot changed the title Update listen.py Update for filters and if from_user is None Jan 24, 2023
@usernein
Copy link
Owner

Hey, thanks for your effort on this huge contribution! I did not understand at first why patchable got those new parameters, but then i realized you may be using it outside pyromod.

I'm going to fix the conflicts, then merge the PR and then merge the changes onto the brand new refactored pyromod v3

Thank you!

@usernein usernein merged commit e3981fe into usernein:master Oct 29, 2023
usernein added a commit that referenced this pull request Oct 29, 2023
* Update listen.py

* Update listen.py

* Update utils.py

* Changes to be committed:
	modified:   pyromod/listen/listen.py
	modified:   pyromod/utils/utils.py

* Changes to be committed:
	modified:   pyromod/utils/utils.py

* Changes to be committed:
	modified:   pyromod/utils/utils.py

* Changes to be committed:
	modified:   pyromod/utils/utils.py

* Update utils.py

* Update utils.py

* Update utils.py

* Update utils.py

* Update utils.py

* Update utils.py

* Update utils.py

* Update utils.py

* Changes to be committed:
	modified:   pyromod/listen/listen.py

* Update pyproject.toml

* Update poetry.lock

* Update poetry.lock

* Update pyproject.toml

* Update listen.py

* Update listen.py

---------

Co-authored-by: Cezar H <pauxiscezar@gmail.com>
usernein added a commit that referenced this pull request Oct 29, 2023
* feat: remove identifier of listen() and ask() in favor of new kwargs

* refactor: split classes into new files, extend from original classes

* refactor: rename patch -> patch_into, patchable -> should_patch

* style: format with ruff

* feat(precedence): return most specific listener

* refactor!: rename get_single_listener -> get_matching_listener, get_many_listeners -> get_many_matching_listeners

* refactor: improve variable names and only remove checking attribute from null listener

* refactor: transform config.py into package pyromod.config

* chore!: remove handlers from pyromod exports

* docs: bring new docs with docsify

* docs: update README

* Create CNAME

* Delete CNAME

* docs: add missing backtick

* Create CNAME

* fix: fix order of positional args in bound method ask

* refactor(ask)!: rename request to sent_message

* feat(ask): send message only if text is not empty

* refactor: search for message_id (pyrogram v1) if message.id is not found

* refactor: only use query.message.chat.id if both message and chat are not null

* supports pyrogram v1 (#11)

* supports pyrogram v1

* Update listen.py

forgot, delete comma

---------

Co-authored-by: Cezar H <pauxiscezar@gmail.com>

* request to sent_message (#13)

I think using the sent_message statement instead of request is much more descriptive

* ask function will behave like listen if text is empty (#14)

Co-authored-by: Cezar H <pauxiscezar@gmail.com>

* Update for filters and if from_user is None (#17)

* Update listen.py

* Update listen.py

* Update utils.py

* Changes to be committed:
	modified:   pyromod/listen/listen.py
	modified:   pyromod/utils/utils.py

* Changes to be committed:
	modified:   pyromod/utils/utils.py

* Changes to be committed:
	modified:   pyromod/utils/utils.py

* Changes to be committed:
	modified:   pyromod/utils/utils.py

* Update utils.py

* Update utils.py

* Update utils.py

* Update utils.py

* Update utils.py

* Update utils.py

* Update utils.py

* Update utils.py

* Changes to be committed:
	modified:   pyromod/listen/listen.py

* Update pyproject.toml

* Update poetry.lock

* Update poetry.lock

* Update pyproject.toml

* Update listen.py

* Update listen.py

---------

Co-authored-by: Cezar H <pauxiscezar@gmail.com>

* feat: remove identifier of listen() and ask() in favor of new kwargs

* refactor: split classes into new files, extend from original classes

* refactor: rename patch -> patch_into, patchable -> should_patch

* refactor: rename patch -> patch_into

* style: format with ruff

* chore: remaking many changes that got undone after the merge

* refactor!: rename get_single_listener -> get_matching_listener, get_many_listeners -> get_many_matching_listeners

* feat(precedence): return most specific listener

* fix: replace @should_patch -> @should_patch()

* fix: use Dict and List from typing instead of builtin

---------

Co-authored-by: Tofik Denianto <77754555+tofikdn@users.noreply.github.com>
Co-authored-by: Eikosa <darkofheking@gmail.com>
Co-authored-by: Jusidama Bot <97734754+Jusidama-Bot@users.noreply.github.com>
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.

None yet

2 participants