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 chat_type_filter.py #504

Merged
merged 2 commits into from
Feb 20, 2021
Merged

Update chat_type_filter.py #504

merged 2 commits into from
Feb 20, 2021

Conversation

sijokun
Copy link

@sijokun sijokun commented Feb 9, 2021

Description

Function comment error

Fixes # (issue)

Type of change

  • Documentation (typos, code examples or any documentation update)

Copy link
Contributor

@evgfilim1 evgfilim1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

Copy link
Contributor

@evgfilim1 evgfilim1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry, haven't noticed a small problem at first

Comment on lines 25 to 26
This handler will be called when user sends message in private chat or channel
"""
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please indent these lines properly

Suggested change
This handler will be called when user sends message in private chat or channel
"""
This handler will be called when user sends message in private chat or channel
"""

@evgfilim1 evgfilim1 added docs Something is missing in docs enhancement Make it better! labels Feb 9, 2021
Copy link
Contributor

@evgfilim1 evgfilim1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good now, thanks for contributing!

@evgfilim1 evgfilim1 merged commit 675bd1c into aiogram:dev-2.x Feb 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Something is missing in docs enhancement Make it better!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants