Skip to content

added the missing "certificate" parameter in the "SetWebhook" - #192

Merged
vjik merged 3 commits into
phptg:masterfrom
userator:fix-certificate-parameter
Apr 5, 2026
Merged

added the missing "certificate" parameter in the "SetWebhook"#192
vjik merged 3 commits into
phptg:masterfrom
userator:fix-certificate-parameter

Conversation

@userator

@userator userator commented Apr 4, 2026

Copy link
Copy Markdown
Contributor

The "certificate" parameter is currently missing in the "SetWebhook" method.
This pull request fixes this issue.

https://core.telegram.org/bots/api#setwebhook

@userator
userator force-pushed the fix-certificate-parameter branch 2 times, most recently from e93cf11 to d5929c8 Compare April 4, 2026 15:27

@vjik vjik left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@userator Good catch!

Add line to changelog and see comment below.

Comment thread src/Method/Update/SetWebhook.php Outdated
Comment thread src/TelegramBotApi.php Outdated
@userator

userator commented Apr 4, 2026

Copy link
Copy Markdown
Contributor Author

line to changelog added

@userator
userator requested a review from vjik April 4, 2026 17:47
@userator
userator force-pushed the fix-certificate-parameter branch 2 times, most recently from 6e86995 to 9ac2474 Compare April 4, 2026 17:50
… `certificate` field to `TelegramBotApi::setWebhook()` method.
@userator
userator force-pushed the fix-certificate-parameter branch from 9ac2474 to 80b7a2a Compare April 4, 2026 17:54
Comment thread CHANGELOG.md Outdated
Comment thread CHANGELOG.md Outdated
@vjik
vjik merged commit 3f60f48 into phptg:master Apr 5, 2026
18 checks passed
@vjik

vjik commented Apr 5, 2026

Copy link
Copy Markdown
Member

@userator Thanks for contribute!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants