Skip to content

Conversation

@oShadow05
Copy link
Contributor

Replace bot.send(chat, message) with
bot.chat(chat).send(message)

Fix typos in the buttons code

Replace bot.send(chat, message) with bot.chat(chat).send(message) Fix typos in the buttons code
Copy link
Contributor

@pietroalbini pietroalbini left a comment

Choose a reason for hiding this comment

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

Thank you! Looks good to me after you remove the "fix" of the callback example.

btns[0].url("Visit example.com", "http://www.example.com")
btns[1].url("example.net", "http://www.example.net")
btns[1].url("example.org", "http://www.example.org")
btns[2].url("example.org", "http://www.example.org")
Copy link
Contributor

Choose a reason for hiding this comment

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

This was intentional, to show off adding multiple buttons in the same line :)

btns = botogram.Buttons()
btns[0].callback("Commit changes", "commit")
btns[0].callback("Checkout master", "checkout", "master")
btns[1].callback("Checkout master", "checkout", "master")
Copy link
Contributor

Choose a reason for hiding this comment

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

Same as above, this is what I wanted.

Copy link
Contributor

@pietroalbini pietroalbini left a comment

Choose a reason for hiding this comment

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

LGTM

@pietroalbini pietroalbini merged commit e9ff044 into python-botogram:master Oct 11, 2017
@pietroalbini
Copy link
Contributor

Thank you again for this fixes! The updated pages should go live on the in-development docs in a few minutes.

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

Labels

None yet

2 participants