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

fix: fix ipns routing test #2203

Closed
wants to merge 1 commit into from
Closed

Conversation

hugomrdias
Copy link
Member

@hugomrdias hugomrdias commented Jun 27, 2019

This test 'should use the dht if enabled' was failing because the ipns routing was trying to get the wrong option.

This test 'should use the dht if enabled' was fail because the ipns routing was trying to get the wrong option.
@hugomrdias hugomrdias requested a review from alanshaw June 27, 2019 09:35
@alanshaw
Copy link
Member

I think @vasco-santos has the correct fix for this one #2205

@alanshaw alanshaw closed this Jun 27, 2019
@vasco-santos vasco-santos deleted the fix/ipns-routing-test branch June 27, 2019 15:58
@hugomrdias
Copy link
Member Author

You sure if you check libp2p config there is not "config" property

@vasco-santos
Copy link
Member

Have you really check the libp2p config?

libp2p/js-libp2p#creating-your-own-libp2p-bundle

@hugomrdias
Copy link
Member Author

Found it in the validation, you are right thx

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