-
Notifications
You must be signed in to change notification settings - Fork 2
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
Client create collection via http client #17
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good work overall! Thanks!
I would also request adding suitable test(s), see other api call related tests
c9525a2
to
f4b8020
Compare
@rsjethani can you please have another look on this so I can continue with the CLI 🙏 🙂 |
f4b8020
to
e897b01
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just noting that this new API does not provide a way to provide optional properties for the new collection. We would need a new issue for that later.
Does not fix anything, but is somewhat part of #14
This adds the support for the HTTP client to create a collection.
https://www.getoutline.com/developers#tag/Collections/paths/~1collections.create/post
Only "name" is required, so I added it.
We can later adjust it. Not sure (yet) what the default value for
private
is.I assume it is
true
, but if not, we should add it to the request and make it later configurable.I also fixed a few typos suggested by my IDE :)