ipfs / pinning-services-api-spec

Standalone, vendor-agnostic Pinning Service API for IPFS ecosystem
https://ipfs.github.io/pinning-services-api-spec/
Creative Commons Zero v1.0 Universal
100 stars 27 forks source link

Remove unpinned status #30

Closed lidel closed 4 years ago

lidel commented 4 years ago

This PR removes unpinned from Status enum.

Rationale

There is no way to end up in "unpinned" state when using this API other than removing a pin object. This implies pinning service is required to keep a history of all past pins in case user wants to bring them back.

After internal reviews this status does not make sense for MVP, is not useful for integration with IPFS Desktop/WebUI If user wants to bring back old Pin (eg. due to its metadata), they can read it before unpinning, persist it off-band and then add a new Pin in the future.

cc @obo20