Skip to content

Conversation

Poolitzer
Copy link
Member

@Poolitzer Poolitzer commented Nov 12, 2022

I am keeping the branch updated with the merge to one :D

Currently implemented:

  • get forum topic icon stickers
  • edit forum topic
  • create forum topic
  • close forum topic
  • delete forum topic
  • reopen forum topic
  • unpin all topic messages

Copy link
Member

@harshil21 harshil21 left a comment

Choose a reason for hiding this comment

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

initial review

@harshil21 harshil21 added the ⚙️ bot-api affected functionality: bot-api label Nov 12, 2022
@harshil21 harshil21 added this to the v20.0a5 milestone Nov 12, 2022
Copy link
Member

@harshil21 harshil21 left a comment

Choose a reason for hiding this comment

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

review 2 (fixing this myself)

@Bibo-Joshi
Copy link
Member

IMO we can merge, then. We'll have to do a final review of #3360 anyway :D

@Bibo-Joshi Bibo-Joshi merged commit addaa6d into api_6.3_update Nov 15, 2022
@Bibo-Joshi Bibo-Joshi deleted the 6.3_new_methods branch November 15, 2022 17:02
@lemontree210
Copy link
Member

I'll mention #3340 here if you don't mind. Helps my mental navigation :)

@lemontree210
Copy link
Member

I know it's merged already so I'm late, but it looks like the params icon_color and icon_custom_emoji_id of method Bot.create_forum_topic are optional in Telegram API but aren't optional in our bot.py?

@github-actions github-actions bot locked and limited conversation to collaborators Nov 23, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
⚙️ bot-api affected functionality: bot-api
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants