Skip to content

Add option to register onion address#355

Closed
ricott1 wants to merge 31 commits intoensdomains:devfrom
ricott1:onionSupport
Closed

Add option to register onion address#355
ricott1 wants to merge 31 commits intoensdomains:devfrom
ricott1:onionSupport

Conversation

@ricott1
Copy link

@ricott1 ricott1 commented Jun 6, 2019

This pull is similar to #328 . I don't know how to update a pull from someone else, so I created a new one.

I added the option to register an onion address, encoded as a multiaddress, in the contenthash entry.

This pull will work in tandem with a metamask pull.

It depends on another pull made on the content-hash dependency.

makoto and others added 18 commits May 3, 2019 14:03
* Add legacy registrant
* Only allow DeedOwner to transfer
Deploy to production - Fix transfer registrant
Deploy to production - Fix 0x00000
Fix legacy auction entry for 0x00000
Deploy to production - fix error
Deploy to production - Fix safari notifications
Deploy to production - Infura new URL
Show parsed search result (#324)
@Arachnid Arachnid requested a review from jefflau July 9, 2019 20:38
@Arachnid
Copy link
Member

Arachnid commented Jul 9, 2019

@jefflau PTAL

@jefflau
Copy link
Member

jefflau commented Jul 10, 2019

Hi @ricott1,

Sorry for the delay at looking at this.

We've moved this functionality in the most recent refactor to https://github.com/ensdomains/ui

Is it possible you could make this pull request on that repo?

@ricott1
Copy link
Author

ricott1 commented Jul 10, 2019

Hi @jefflau ,
Do you mean to include the possibility to encode onion in the src/utils/contents.js file?

@ricott1
Copy link
Author

ricott1 commented Jul 16, 2019

Closing, PR now at ensdomains/ui#13

@ricott1 ricott1 closed this Jul 16, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

Comments