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

Added group name #47

Merged
merged 7 commits into from
Mar 3, 2016
Merged

Added group name #47

merged 7 commits into from
Mar 3, 2016

Conversation

RichardLitt
Copy link
Contributor

I was unable to get an argument working with resolve. That should work.

@@ -1661,9 +1661,261 @@ root will not be listable, as it is virtual. Access known paths directly.

# Group name

## publish
IPFS namespace (IPNS) tool
Copy link
Collaborator

Choose a reason for hiding this comment

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

Link to ipns spec/description/docs would be good

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Not sure about that. I'd rather not add links which aren't part of the CLI mans.

@RichardLitt
Copy link
Contributor Author

@dignifiedquire added a time type. Thoughts?

@RichardLitt RichardLitt force-pushed the feature/name branch 2 times, most recently from f1d026b to 6538ebb Compare February 19, 2016 18:22
@RichardLitt
Copy link
Contributor Author

Note that resolve defaults are not yet in the IPFS API. Holding off on PRing until fixed ipfs/kubo#2424

time formats can be joined. For example, '3h30m'.

+ (string) - The time format, composed of a number followed by an enumerated list with the following members.
+ Members
Copy link
Collaborator

Choose a reason for hiding this comment

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

If it's not an enum Members is wrong afaik.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Not sure how else to do this. Just a text description?

Copy link
Collaborator

Choose a reason for hiding this comment

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

yeah

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Updated.

@dignifiedquire
Copy link
Collaborator

LGTM

RichardLitt added a commit that referenced this pull request Mar 3, 2016
@RichardLitt RichardLitt merged commit e8e9bba into master Mar 3, 2016
@RichardLitt RichardLitt deleted the feature/name branch March 3, 2016 20:54
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.

2 participants