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

rpc/nns: support NEP-18 addresses #392

Merged
merged 1 commit into from
Jun 18, 2024
Merged

rpc/nns: support NEP-18 addresses #392

merged 1 commit into from
Jun 18, 2024

Conversation

roman-khimov
Copy link
Member

No description provided.

@roman-khimov roman-khimov added this to the v0.20.0 milestone Mar 13, 2024
AnnaShaleva
AnnaShaleva previously approved these changes Mar 13, 2024
cthulhu-rider
cthulhu-rider previously approved these changes Mar 14, 2024
Copy link
Contributor

@cthulhu-rider cthulhu-rider left a comment

Choose a reason for hiding this comment

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

code ok

@roman-khimov describe motivation in the commit pls

@roman-khimov
Copy link
Member Author

Let's wait for neo-project/proposals#133 to finally settle, but standards are good because they're standards, any compliant tooling can handle the data then.

carpawell
carpawell previously approved these changes Mar 22, 2024
@roman-khimov roman-khimov added the blocked Can't be done because of something label Mar 22, 2024
@roman-khimov roman-khimov removed the blocked Can't be done because of something label Jun 17, 2024
@roman-khimov
Copy link
Member Author

Rebased to fix the conflict and it can be merged now, NEP-18 is official.

@roman-khimov
Copy link
Member Author

Test failure is subject to #402, this one doesn't touch contracts.

@roman-khimov roman-khimov merged commit 53fb384 into master Jun 18, 2024
7 of 10 checks passed
@roman-khimov roman-khimov deleted the nep-18-addresses branch June 18, 2024 08:19
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