Skip to content
This repository has been archived by the owner on Mar 10, 2020. It is now read-only.

Add pin option to dag.put #456

Closed
joeltg opened this issue Apr 4, 2019 · 1 comment
Closed

Add pin option to dag.put #456

joeltg opened this issue Apr 4, 2019 · 1 comment

Comments

@joeltg
Copy link

joeltg commented Apr 4, 2019

Right now you can stick a boolean pin property in the options argument to ipfs.add (and addReadableStream, etc), but there's no equivalent for ipfs.dag.put. This is awkward if you want to selectively pin or not pin your IPLD nodes and amounts to two-step ipfs.dag.put(object, options).then(cid => shouldWePinThis && ipfs.pin.add(cid.toString()) construction.

Not sure if this is the appropriate place to raise this issue but thought I'd bring it up here.

@alanshaw
Copy link
Contributor

This was implemented in ipfs/js-ipfs#2521

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

No branches or pull requests

2 participants