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 ChatBoostSource issues #2138

Merged
merged 2 commits into from
Jan 26, 2024
Merged

Fix ChatBoostSource issues #2138

merged 2 commits into from
Jan 26, 2024

Conversation

Badiboy
Copy link
Collaborator

@Badiboy Badiboy commented Jan 21, 2024

Description

Fix ChatBoostSource issues

  1. "source" is temporary made optional
  2. source types are parsed correctly

1. "source" is temporary made optional
2. source types are parsed correctly
@coder2020official
Copy link
Collaborator

LGTM, but let's wait for the answer from bot API dev

@Badiboy
Copy link
Collaborator Author

Badiboy commented Jan 21, 2024

but let's wait for the answer from bot API dev

By making source "Optional" we do not break anything, just fix the current issue for users. When we got answer - we cann roll it back, it's not a problem :)

@coder2020official
Copy link
Collaborator

coder2020official commented Jan 21, 2024

https://github.com/tdlib/telegram-bot-api/blob/ade0841d41b7126c2d908475291a5e1cd8a1b905/telegram-bot-api/Client.cpp#L3394
It is not optional; I doubt that it is an API bug
We have to investigate the issue further

@Badiboy
Copy link
Collaborator Author

Badiboy commented Jan 26, 2024

It's wrong to wait more.

@coder2020official When something will change - will make an update.

@Badiboy Badiboy merged commit d5e5269 into eternnoir:master Jan 26, 2024
7 checks passed
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.

2 participants