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
Support remote files API #859
Conversation
fe729c4
to
8ccbda4
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.
Thank you!
Please add test case 🙏
8ccbda4
to
ecbe29c
Compare
ecbe29c
to
46dc7e1
Compare
56feeb9
to
3eb3ab3
Compare
@kanata2 Added tests for remotefiles module. |
@kanata2 |
Sorry for slow response. I'll confirm in the next few days... |
} | ||
|
||
// AddRemoteFile adds a remote file | ||
func (api *Client) AddRemoteFile(params RemoteFileParameters) (remotefile *RemoteFile, err error) { |
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.
func (api *Client) AddRemoteFile(params RemoteFileParameters) (remotefile *RemoteFile, err error) { | |
func (api *Client) AddRemoteFile(params RemoteFileParameters) (*RemoteFile, error) { |
} | ||
|
||
// ListRemoteFiles retrieves all remote files according to the parameters given. Uses cursor based pagination. | ||
func (api *Client) ListRemoteFiles(params ListRemoteFilesParameters) ([]RemoteFile, *ListRemoteFilesParameters, error) { |
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.
How about using string instead of ListRemoteFilesParameters
in returned value?
I think ListRemoteFilesParameters
should be used only as an argument to ListRemoteFiles.
Please add documentation link to each GoDoc comment 🙏 |
This PR supports remote files APIs (#856)
I implemented remotefiles module referencing files module.