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

Fix types.MediaGroup methods #514

Merged
merged 1 commit into from
Mar 14, 2021
Merged

Fix types.MediaGroup methods #514

merged 1 commit into from
Mar 14, 2021

Conversation

WhiteMemory99
Copy link
Contributor

@WhiteMemory99 WhiteMemory99 commented Feb 27, 2021

Description

The methods of MediaGroup were missing some parameters, type hints and docstrings. This PR fixes that along with all the mess.

Type of change

  • Documentation (typos, code examples or any documentation update)
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • New and existing unit tests pass locally with my changes

@evgfilim1 evgfilim1 added enhancement Make it better! bug Something is wrong with the framework docs Something is missing in docs labels Feb 27, 2021
@JrooTJunior JrooTJunior merged commit 5254329 into aiogram:dev-2.x Mar 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something is wrong with the framework docs Something is missing in docs enhancement Make it better!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants