[SendPhoto]: 400 Bad request (chat_id not found)

[bekzod-murotboyev]

Describe the bug
org.telegram.telegrambots.meta.api.methods.send.SendPhoto class fields are not annotated with @JsonProperty annotation, that is why always telegram bot api returns 400 Bad request with content of "chat_id" not found!

To Reproduce
Steps to reproduce the behavior:

1. Go to org.telegram.telegrambots.meta.api.methods.send.SendPhoto
2. Annotate chatId and other fields with @JsonProperty annotation

[Georglider]

I have made a pull request regarding to this issue: #1302

[TheQuiu]

🎂👏💋

[bekzod-murotboyev]

Thanks for your attention. From which version those changes will be available?

[Georglider]

Thanks for your attention. From which version those changes will be available?

I don't know, owner of this repo should accept #1302 in order for changes to apply.

[rubenlagus]

I'll check it now that the new version is fully out. Need to consider this change carefully since this was intentionally done as it is now (since the proposed solution is only partial support for the Bots API. hence, would require some extra documentation/Javadocs to ensure it is clear for everything.

Also, the proposed MR doesn't include any kind of UT that proves it works as it is expected

[stokado]

@rubenlagus Hello!
Are there any plans to merge @Georglider MR?
Telegram Docs state that we should pass chat_id field for destination, not chatId