Poolitzer · GitHub

@Poolitzer

Copy link Copy Markdown

Member

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
Also doing black a solid here

harshil21

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.

initial review

harshil21

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.

review 2 (fixing this myself)

new constant class also renders the topic colors as a box
as its a subset of test_forum in branch new_classes_for_api_6.3

Bibo-Joshi

@Bibo-Joshi

Copy link Copy Markdown

Member

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

@Bibo-Joshi

@Bibo-Joshi

@lemontree210

Copy link Copy Markdown

Member

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

@lemontree210

Copy link Copy Markdown

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?

Read the original on github.com ↗