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

ENS metadata and reverse ENS record set #2116

Merged
merged 6 commits into from
Dec 3, 2024
Merged

ENS metadata and reverse ENS record set #2116

merged 6 commits into from
Dec 3, 2024

Conversation

NickSneo
Copy link
Contributor

@NickSneo NickSneo commented Nov 25, 2024

What does this PR do?

Adds retrieving ENS metadata and lets user set reverse ENS record

Where should the reviewer start?

all

Why is it needed?

Functionality was not available

Checklist

  • I've read the contribution guidelines.
  • I've added tests (if applicable).
  • I've added a changelog entry if necessary.

Signed-off-by: Nischal Sharma <nischal@web3labs.com>
Signed-off-by: Nischal Sharma <nischal@web3labs.com>
Signed-off-by: Nischal Sharma <nischal@web3labs.com>
Signed-off-by: Nischal Sharma <nischal@web3labs.com>
Signed-off-by: Nischal Sharma <nischal@web3labs.com>
Copy link
Contributor

@gtebrean gtebrean left a comment

Choose a reason for hiding this comment

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

few suggestions

core/src/main/java/org/web3j/ens/NameWrapperApi.java Outdated Show resolved Hide resolved
* <p>Generated with web3j version 4.12.2.
*/
@SuppressWarnings("rawtypes")
public class ReverseRegistrar extends Contract {
Copy link
Contributor

Choose a reason for hiding this comment

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

is there any possibility to generate this wrapper when we need it, as we are doing for the integration tests in order to avoid this hardcode here

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I dont it think it will be possible, the same logic is done for ENS, ENS registry and ENS offchain resolver contracts. As these classes need to be present in compile time and included in the JAR.

Signed-off-by: Nischal Sharma <nischal@web3labs.com>
@NickSneo NickSneo merged commit 1c68afd into main Dec 3, 2024
5 checks passed
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.

2 participants