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

Added group get #46

Merged
merged 8 commits into from
Apr 19, 2016
Merged

Added group get #46

merged 8 commits into from
Apr 19, 2016

Conversation

RichardLitt
Copy link
Contributor

@RichardLitt RichardLitt commented Feb 6, 2016

Bugs:

  • I was unable to get archive to work
  • Archive and compression options change the type of call; the first is GET, the second POST


+ Parameters
+ arg (string, required) - The path to the IPFS object or objects to be outputted
+ output (string, optional) - The path where output should be stored. Default: ./<ipfs-path>.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • where the output should be stored.
  • Shouldn't the default value be in backticks?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done.

@RichardLitt
Copy link
Contributor Author

@daviddias
Copy link
Contributor

Still with problems on Archive?

Bugs:

- [ ] I was unable to get archive to work
- [ ] Archive and compression options change the type of call; the first is GET, the second POST
@RichardLitt RichardLitt merged commit 7926fd7 into master Apr 19, 2016
@RichardLitt
Copy link
Contributor Author

I am confident that it is better that this is merged than the alternative.

@RichardLitt RichardLitt deleted the feature/get branch April 19, 2016 15:30
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants