Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixes #126 #133

Merged
merged 2 commits into from
May 5, 2017
Merged

Fixes #126 #133

merged 2 commits into from
May 5, 2017

Conversation

madsmtm
Copy link
Member

@madsmtm madsmtm commented May 5, 2017

Fixes #126
Updated readme, and changed getThreadInfo, so last_n actually specifies the amount of retrieved messages

@carpedm20
Copy link
Collaborator

Yes the number should be bigger than 0. Thanks!

@carpedm20 carpedm20 closed this May 5, 2017
@carpedm20 carpedm20 reopened this May 5, 2017
@carpedm20 carpedm20 merged commit 921fbb4 into fbchat-dev:master May 5, 2017
@carpedm20
Copy link
Collaborator

@madsmtm Hi. I think you have a great interest on project. Nowadays I am fully focused on machine learning research so if you are interested in being a Collaborator (getting push access to the repo) for this repo, feel free to tell me.

@madsmtm
Copy link
Member Author

madsmtm commented May 8, 2017

Hi @carpedm20. I do have great interest in this project ;). Being able to help out even more where I can would be lovely! Hope you enjoy your research!

@carpedm20
Copy link
Collaborator

@madsmtm Thanks! Thanks for the all contribution you did and I hope you enjoy this project.

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.

2 participants