Skip to content
This repository has been archived by the owner on Aug 15, 2022. It is now read-only.

Fix bot output when it's to a user #95

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

awentzonline
Copy link

This seems to return a dict rather than a string. Looks like there was a similar fix for group messaging here: bba41f4#diff-933761be909a15a181706d770d010687

@CLAassistant
Copy link

CLAassistant commented Nov 3, 2018

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@aoberoi
Copy link
Contributor

aoberoi commented Dec 12, 2018

@awentzonline thanks for the contribution! two things:

  1. we'll need you to sign the CLA before we can merge this.

  2. travis is complaining about core.py:7:1: F401 'json' imported but unused. i think we just need to remove the import json in that file. would you mind doing that please?

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 this pull request may close these issues.

3 participants