Skip to content
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

add ReverseRegistration event #26

Closed
wants to merge 3 commits into from

Conversation

scottrepreneur
Copy link

No description provided.

Copy link
Member

@Arachnid Arachnid left a comment

Choose a reason for hiding this comment

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

Thanks for this! If we're adding support for reverse names, though, we should cover all the cases - can you add handlers for claim/claimWithResolver too?

@frolic
Copy link

frolic commented Jun 24, 2022

I'm interested in reviving this PR so that it can be merged.

@Arachnid Can you share more about what info you're looking to index in the claim and claimWithResolver calls? Is there something missing here for compatibility purposes?

@Arachnid
Copy link
Member

Arachnid commented Oct 2, 2022

The new reverse registrar, currently deployed on testnets, will support emitting the appropriate events - so we will need to write support for that rather than doing transaction tracing like this.

@Arachnid Arachnid closed this Oct 2, 2022
@Dizotoff
Copy link

Dizotoff commented Dec 5, 2022

Hi @Arachnid

Is there a way to determine the primary ENS name by wallet address at the moment?

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.

5 participants