-
Notifications
You must be signed in to change notification settings - Fork 37
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
fix(contracts): add and remove operator refactor #358
Conversation
e8bdf35
to
7116d43
Compare
a1a917e
to
41e089c
Compare
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.
LGTM. Could you rename flags and cmd/validator.go as well?
align event processing with smart contract changes issue: closes #393
cd1276b
to
248a978
Compare
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.
Could you rename flags and cmd/validator.go as well?
I will handle this. LGTM
Binary uploaded successfully 🎉📦 Version Name: 0.14.1-unstable-3025bfb |
According to the changes of #358, rename the command for set/unset operator. issue: none
Refactor operator methods to reflect there is only 1 operator to set and unset
issue: fixes #302