Skip to content

Conversation

@EricDahlvang
Copy link
Member

Fixes #1559

(CoreBot will be a separate PR, since there are existing changes in a different branch)

Copy link
Contributor

@JonathanFingold JonathanFingold left a comment

Choose a reason for hiding this comment

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

@EricDahlvang Looks good. Please don't merge yet. I need to update the topics that reference these code snippets. I'll let you know when the corresponding doc PR is ready, and we can merge them on the same day in their respective repos.

@JonathanFingold
Copy link
Contributor

@EricDahlvang , I've created the bot-docs-pr PR. Go ahead and merge this, and I'll have the docs PR merged today, too.

@EricDahlvang EricDahlvang merged commit ba84368 into master Jun 17, 2019
@EricDahlvang EricDahlvang deleted the eric/removeCreateReply branch June 17, 2019 17:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

C#: Do not use Activity.CreateReply in any sample code

3 participants