Skip to content
This repository has been archived by the owner on Sep 20, 2024. It is now read-only.

Typo in onChange example #1879

Closed
mattcobb opened this issue Dec 19, 2019 · 0 comments · Fixed by #1880
Closed

Typo in onChange example #1879

mattcobb opened this issue Dec 19, 2019 · 0 comments · Fixed by #1880

Comments

@mattcobb
Copy link

What are you trying to achieve or the steps to reproduce?

Fixing a typo in the onChange() example comments in packages/botkit/src/conversation.ts

     * convo.onChange('name', async(response, convo, bot) {

should be

     * convo.onChange('name', async(response, convo, bot) => {

What was the result you received?

When the example is copied into a program, it has a syntax error.

What did you expect?

Syntactically correct example

Context:

  • Botkit version: 4.5
  • Messaging Platform: msteams
  • Node version: 10.15
  • Os: Mac & Linux
  • Any other relevant information:
    I made this fix local and I'll create a PR for it
mattcobb pushed a commit to mattcobb/botkit that referenced this issue Dec 19, 2019
benbrown added a commit that referenced this issue Dec 20, 2019
Fixes #1879 Typo in onChange() example
deepakmahakale pushed a commit to deepakmahakale/botkit that referenced this issue Oct 5, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant