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

fix: ipns using offline datastore if no dht #1988

Closed

Conversation

vasco-santos
Copy link
Member

@vasco-santos vasco-santos commented Apr 11, 2019

When the DHT is not enabled, we must add the OfflineDatastore to for IPNS instead of the DHT

Copy link
Contributor

@dirkmc dirkmc 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
Copy link
Member

Can we get the build green and add a test?

@vasco-santos vasco-santos force-pushed the fix/ipns-using-offline-datastore-if-no-dht branch from 7eb8937 to 8617d65 Compare April 12, 2019 09:26
@alanshaw
Copy link
Member

@vasco-santos I've merged this into #1991 as I think they now depend on each other!

@vasco-santos
Copy link
Member Author

@alanshaw just added a new commit to skip the ipns over dht tests. You should grab it as well, or skip in your PR.

@alanshaw alanshaw closed this Apr 12, 2019
@ghost ghost removed the status/in-progress In progress label Apr 12, 2019
@alanshaw alanshaw deleted the fix/ipns-using-offline-datastore-if-no-dht branch April 12, 2019 13:31
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.

3 participants