Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Rename BusinessBotRights.can_delete_sent_messages to BusinessBotRights.can_delete_outgoing_messages #4844

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

Closed
wants to merge 1 commit into from

Conversation

Bibo-Joshi
Copy link
Member

Based on https://t.me/bot_api_changes/244
Awaiting confirmation from manual testing by @harshil

Note that I did not make this change backward compatible. Our stability policy does say that we keep API functionality available for the current and the next major version of PTB or until the next version of the Bot API. However, as this change is more of a Bot API bug-fix without proper deprecation note or even changelog, I find it hard to put it into this category. Please let me know if you see an issue with this.

@Bibo-Joshi Bibo-Joshi added ⚙️ bot-api affected functionality: bot-api 🛠 breaking change type: breaking 📋 do-not-merge-yet work status: do-not-merge-yet labels Jul 2, 2025
@Bibo-Joshi
Copy link
Member Author

Testing by @harshil21 shows that can_delete_sent_messages is returned. Waiting for clarification via tdlib/telegram-bot-api#758.

@Bibo-Joshi
Copy link
Member Author

Apparently the change was accidental. Closing.

@Bibo-Joshi Bibo-Joshi closed this Jul 3, 2025
@Bibo-Joshi Bibo-Joshi deleted the rename-bot-rights branch July 3, 2025 04:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⚙️ bot-api affected functionality: bot-api 🛠 breaking change type: breaking 📋 do-not-merge-yet work status: do-not-merge-yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant