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

box_add_description() should be box_file_update() #169

Closed
ijlyttle opened this issue Aug 22, 2020 · 3 comments
Closed

box_add_description() should be box_file_update() #169

ijlyttle opened this issue Aug 22, 2020 · 3 comments

Comments

@ijlyttle
Copy link
Member

ijlyttle commented Aug 22, 2020

The Box API offers a lot of things with their file-update endpoint - I think this function could be expanded and renamed.

To be clear, I am not suggesting we do this on this (Aug. 2020) release, but perhaps soon thereafter.

@ijlyttle
Copy link
Member Author

ijlyttle commented Sep 19, 2020

We have a lot of open issues that seem to hit the same API endpoint: File Update:

We already have an internal function box_file_update() - used by box_ul()

We also have an incidental issue, #174 on file metadata, potentially related, as it concerns the Box API file object.

Not considering immediate action other than possibly closing #182 and #167, combining them into this issue.

@nathancday
Copy link
Member

I've personally never used the boxr_add_description() once I found out it wasn't a comment, non-API users could see. It does feel like we are missing some functionality and agree about expanding to support the lock/copy/link/move behaviors.

@ijlyttle
Copy link
Member Author

closing in favor of #208

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

No branches or pull requests

2 participants