Skip to content

Integrate RNS Registry into EnsController #5973

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

jonathansmirnoff
Copy link

Explanation

This PR adds the RNS Registry address to the EnsController package. This enhancement enables the resolution of .rsk domains in the Rootstock-compatible version of MetaMask.

Why:

By including the RNS Registry address, users will be able to resolve human-readable domain names (e.g., alice.rsk) directly in MetaMask when interacting with dApps built on Rootstock. This improves the user experience and provides native support for domain name resolution within the Rootstock ecosystem.

Note:

We are aware that additional changes are required in MetaMask to fully support RNS resolution. However, this PR represents the first step toward enabling complete RNS integration within the wallet.

Further Reading:

Checklist

  • I've updated the test suite for new or updated code as appropriate
  • I've updated documentation (JSDoc, Markdown, etc.) for new or updated code as appropriate
  • I've communicated my changes to consumers by updating changelogs for packages I've changed, highlighting breaking changes as necessary
  • I've prepared draft pull requests for clients and consumer packages to resolve any breaking changes

@jonathansmirnoff jonathansmirnoff requested review from a team as code owners June 12, 2025 19:21
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.

1 participant