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

fix: stub out call to fetch for ipfs.dns test in browser #1512

Merged
merged 2 commits into from
Aug 16, 2018

Conversation

alanshaw
Copy link
Member

@alanshaw alanshaw commented Aug 16, 2018

Stubs self.fetch to return a static CID for calls to https://ipfs.io/api/v0/dns?arg=ipfs.io.

Removes dependency on external service.

Stubs self.fetch to return a static CID for calls to https://ipfs.io/api/v0/dns?arg=ipfs.io.

Removes dependency on external service.

License: MIT
Signed-off-by: Alan Shaw <alan@tableflip.io>
@ghost ghost assigned alanshaw Aug 16, 2018
@ghost ghost added the status/in-progress In progress label Aug 16, 2018
})
})
}
return _fetch.apply(this, arguments)
Copy link
Member

Choose a reason for hiding this comment

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

Can _fetch be null 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.

yes, better now?

So no chance of null is not a function error.

License: MIT
Signed-off-by: Alan Shaw <alan@tableflip.io>
Copy link
Member

@achingbrain achingbrain left a comment

Choose a reason for hiding this comment

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

LGTM

@alanshaw alanshaw merged commit 86c3d81 into master Aug 16, 2018
@ghost ghost removed the status/in-progress In progress label Aug 16, 2018
@alanshaw alanshaw deleted the fix/stub-fetch-for-dns-in-browser branch August 16, 2018 17:12
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