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

JsonProperty fix for Webhook #1302

Open
wants to merge 17 commits into
base: dev
Choose a base branch
from
Open

Conversation

Georglider
Copy link

Annotated fields with missing JsonProperty in org/telegram/telegrambots/meta/api/methods the way it is done in other classes in the same package.
It helps Webhook to return a correctly serialized json result.

Georglider and others added 6 commits November 16, 2023 21:19
…r classes in the same package.

It helps Webhook to return a correctly serialized json result.
…elegrambots/meta/api/methods the way it is done in other classes in the same package.

It helps Webhook to return a correctly serialized json result.
@rubenlagus
Copy link
Owner

How will you be returning the images/documents in the json?

A semicolon were missing in the Standalone reply code.
The reply signature changed and needs a BiConsumer<BaseAbilityBot, Update> instead of Consumer<Update> in the Ability reply
@Georglider
Copy link
Author

According to Telegram API Docs, you can send InputFile using its file_id or its URL. This works well in a Webhook-designed bot. I encountered this exact issue when was trying to send a photo and MediaGroups this way, so after I made the commit allowing me to return correctly serialized json, my problem was finally fixed and I could finally put a direct link into the InputFile. After that, Telegram successfully responded with a message containing the image or album.

Chase22 and others added 8 commits February 2, 2024 23:44
…ed from 'Boolean' to 'String' (according to TelegramBotAPI Official Documentation). (rubenlagus#1315)
* fix force=true for WriteAccessAllowed

* + test and fixtures on write access allowed
* Fix path for GetMyName method

* Add missing sticker methods from API 6.6

* Fix possible NPE in InputSticker validation

* Add ReplyParameters to SendInvoice API method

* Removed mysterious fields in SetChatTitle
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.

9 participants