Skip to content

Fix missing InlineQueryResultPhoto parse_mode field #331

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
May 13, 2020
Merged

Fix missing InlineQueryResultPhoto parse_mode field #331

merged 1 commit into from
May 13, 2020

Conversation

igorsereda
Copy link

@igorsereda igorsereda commented May 11, 2020 •
edited
Loading

Description

Object InlineQueryResultPhoto did not have field parse_mode while it is present in the api object description (https://core.telegram.org/bots/api#inlinequeryresultphoto).

Type of change

  • Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

Manually

Test Configuration:

  • Operating System: Ubuntu 20.04
  • Python version: 3.8.2

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
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes

@evgfilim1 evgfilim1 added the bug Something is wrong with the framework label May 13, 2020
@JrooTJunior JrooTJunior merged commit 52ebc4b into aiogram:dev-2.x May 13, 2020
@evgfilim1
Copy link
Contributor

@igorsereda Thank you for contribution!

@igorsereda igorsereda deleted the missing-parse-mode branch May 13, 2020 11:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@JrooTJunior JrooTJunior JrooTJunior approved these changes

@evgfilim1 evgfilim1 evgfilim1 approved these changes

Assignees
No one assigned
Labels
bug Something is wrong with the framework
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@igorsereda @evgfilim1 @JrooTJunior