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

Add a method to delete a Container? #79

Open
hideya opened this issue Jul 15, 2015 · 3 comments
Open

Add a method to delete a Container? #79

hideya opened this issue Jul 15, 2015 · 3 comments

Comments

@hideya
Copy link

hideya commented Jul 15, 2015

Maybe I'm overlooking but I don't find any way to delete a Container. I'd expect Container class to have destroy method, but I don't find any. How can I delete a Container (= remove the directory on the server that is associated with the container) ?

@raymondfeng
Copy link
Contributor

The backend has the following REST api to delete a container:

https://github.com/strongloop/loopback-component-storage/blob/master/lib/storage-service.js#L277-L283

It should be mapped to DELETE /api/containers/<container>

@hideya
Copy link
Author

hideya commented Jul 15, 2015

Thanks, @raymondfeng. I see. I guess the SDK just haven't implemented the destroy method that is mapped to the REST API. I think it better be fixed.

@bajtos bajtos changed the title No method to delete a Container? Add a method to delete a Container? Sep 8, 2015
@bajtos
Copy link
Contributor

bajtos commented Sep 8, 2015

👍

I agree the destroy method should be added to the API.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

7 participants