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

request to sent_message #13

Merged
merged 1 commit into from
Oct 29, 2023
Merged

request to sent_message #13

merged 1 commit into from
Oct 29, 2023

Conversation

Eikosa
Copy link
Contributor

@Eikosa Eikosa commented Jan 21, 2023

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

I think using the sent_message statement instead of request is much more descriptive
@usernein usernein merged commit 5fffe45 into usernein:master Oct 29, 2023
@usernein
Copy link
Owner

Thanks for your contribution! It's going to be added on pyromod v3 release, which is coming soon

usernein pushed a commit that referenced this pull request Oct 29, 2023
I think using the sent_message statement instead of request is much more descriptive
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 <[email protected]>

* 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 <[email protected]>

* 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 <[email protected]>

* 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 <[email protected]>
Co-authored-by: Eikosa <[email protected]>
Co-authored-by: Jusidama Bot <[email protected]>
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.

2 participants