Skip to content

Conversation

@Aditya-Rajgor
Copy link
Contributor

Added a simple note on inlinebot.py python file in the examples directory.

I found that it is necessary to enable inline commands for your bot from BotFather from the documentation
here - https://core.telegram.org/bots/inline

I hope it serves some value to the community.

It is necessary to enable inline commands from botfather to use inline commands of your bot, Otherwise it won't work.
To use inline commands, you need to enable it from BotFather, just send `/setinline` and select your bot and then add placeholder text for your bot. There you go!
Copy link
Member

@Bibo-Joshi Bibo-Joshi left a comment

Choose a reason for hiding this comment

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

Hi. Thanks for PRing! While we do point to this in the readme already, there's surely no harm in repeating it in the example.

Comment on lines 6 to 9
IMPORTANT -
To enable inline commands for your bot, you have to send `/setinline` command to `botfather`.
Then select your bot and set placeholder for your bot.
Read more here - https://core.telegram.org/bots/inline
Copy link
Member

Choose a reason for hiding this comment

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

IMO we don't need to be quite as verbose. A simple "Dont forget to enable inline mode with @Botfather - " should suffice imo

Copy link
Contributor Author

Choose a reason for hiding this comment

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

True, just pointing out in the example would be sufficient

Copy link
Member

Choose a reason for hiding this comment

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

would you like to rephrase the note?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sure, Should I rephrase and make PR again? Or you can edit it before accepting it?

Copy link
Member

Choose a reason for hiding this comment

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

you can just make a new commit to your master branch - the PR will automatically be updated

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Updated!

@harshil21 harshil21 added ⚙️ documentation affected functionality: documentation ⚙️ examples affected functionality: examples labels May 26, 2022
@harshil21 harshil21 added this to the v20.0a1 milestone May 26, 2022
@Bibo-Joshi Bibo-Joshi changed the base branch from master to doc-fixes May 29, 2022 13:49
@Bibo-Joshi Bibo-Joshi merged commit d631918 into python-telegram-bot:doc-fixes May 29, 2022
@Bibo-Joshi
Copy link
Member

Thank you for the contribution!

@Bibo-Joshi Bibo-Joshi mentioned this pull request May 29, 2022
5 tasks
@github-actions github-actions bot locked and limited conversation to collaborators Jun 5, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

⚙️ documentation affected functionality: documentation ⚙️ examples affected functionality: examples

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants