Skip to content
This repository has been archived by the owner on Feb 12, 2024. It is now read-only.

Add /api/v0/config/show endpoint #74

Merged
merged 1 commit into from
Feb 28, 2016
Merged

Conversation

fbaiodias
Copy link
Member

Implements the /api/v0/config/show as specified at ipfs-inactive/http-api-spec#29

Related to #49 & very similar to #72

@daviddias daviddias mentioned this pull request Feb 27, 2016
14 tasks
@@ -221,5 +234,15 @@ describe('config', () => {
})
})
})

describe('ipfs.config.show', () => {
Copy link
Member

Choose a reason for hiding this comment

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

why a describe here?

Copy link
Member Author

Choose a reason for hiding this comment

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

Was just trying to keep the structure of the other endpoints' tests, removed both of them

@daviddias
Copy link
Member

overall LGTM :) Let's just take out the unnecessary describe

@fbaiodias fbaiodias force-pushed the feature/http-api-config-show branch from 5a35816 to ca06e29 Compare February 28, 2016 14:43
@daviddias
Copy link
Member

woot! :D

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

Successfully merging this pull request may close these issues.

2 participants