8000 added options to the get room history · Pull Request #17 · Pipoline/rocket-python · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

added options to the get room history #17

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

Merged
merged 13 commits into from Jun 15, 2017
Merged

added options to the get room history #17

merged 13 commits into from Jun 15, 2017

Conversation

ghost
Copy link
@ghost ghost commented Jun 12, 2017

Needed for a project the latest option of the Rocketchat api and used the case to add all missing features. Couldn't verifiy if the "unreads" option works.

@dstegelman
Copy link
Collaborator

Hello and thanks for your contribution @StielEiche .

Have you tested the implementation to make sure it works? I no longer have a real-world server setup to test the APIs. The only thing that makes me slightly nervous is the handling of oldest.

@ghost
Copy link
Author
ghost commented Jun 13, 2017

inclusive and unreads doesn't produce an error nor do they return something, the rest works as intended. I'm not sure why these two don't have an effect. oldest works just normal. I removed the if-else block because your code never used the else part. I wrote you an E-mail with the information for a server where you can test the changes.

@dstegelman
Copy link
Collaborator

@StielEiche Thanks!

@dstegelman dstegelman merged commit 4884bd3 into Pipoline:master Jun 15, 2017
@dstegelman
Copy link
Collaborator

Released in 1.2.3

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
0