-
Notifications
You must be signed in to change notification settings - Fork 100
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 tokens tracker #671
add tokens tracker #671
Conversation
…eo-modules into merge-nep11tracker
…eo-modules into merge-nep11tracker
I think it should be a single plugin. |
It's already a single plugin. No need to combine class of Nep11 and Nep17 together. |
Both |
@Ashuaidehao I've added stub classes for you. Please move the methods into the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested OK. @erikzhang @shargon Ready for final review.
Need your approve. @shargon |
Close #542 and #651 .