-
Notifications
You must be signed in to change notification settings - Fork 38
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
Implement v1 API #9
Comments
@rynefagin Take a look at the above, figure we might create an issue and track our progress via it. |
Sounds good, just what we needed. Thanks @graywolf336 |
@rynefagin I'm going to start working on the chat apis, you good with that? |
Yes |
@rynefagin I am adding pagination to the Rocket.Chat REST API for things like |
@graywolf336 Hmmm. One thought is to use the link header in the response which shifts the burden off of the java api to build links to the next set of results and means we don't have to include pagination data in the payload. |
@rynefagin Sadly that requirement wasn't in the time frame I was allocated to build the pagination support and will be added in the future 👎 |
@graywolf336 any ideas? So pagination will be provided in the body? |
@rynefagin After doing some work on the rest api and pagination, only a few items are going to have them and those are the ones which list everything like the
Does that make sense? |
@rynefagin You can an invite to collaborate on this repository, click here to see the invite. I only say this, as I would like to start making reviews on pull requests required for your review to be considered official you have to be a collaborator on this repository. |
Are there news about implementing v1? |
Is the API compatible with the current version of RocketChat? |
yes it is |
Working on an implementation of the Settings, see https://github.com/col-panic/rocket-chat-rest-client/tree/settingsImpl for current progress. Currently building and testing seems not possible - see #33 |
Right now the following items need to be built out, this will serve as a living issue. When building them out, we need to be sure to build out the complete model with getters and setters along with the method on the main client.
Misc
Authentication
Chat Messages
Users
Channels
Groups
Settings
Integrations
Livechat
The text was updated successfully, but these errors were encountered: