-
Notifications
You must be signed in to change notification settings - Fork 39
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
Neo Integration #124
base: master
Are you sure you want to change the base?
Neo Integration #124
Conversation
In order to resolve the merge conflicts & test this, I rebased the branch as william/rebased-neo-plugin. I also had to create a matching GUI branch at william/add-neo to actually be able to use this plugin. When I run the plugin on those branches, my wallet does not display a balance even though it has funds. The error I see in the console is |
@swansontec can you please try #154 ? We have fixed the issue that you mentioned above. |
@@ -0,0 +1,47 @@ | |||
export const NeoTransactonOnline = { |
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.
Typo in the variable name is causing tests to fail. Should be 'NeoTransactionOnline'
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.
Let's move to #154
Mostly I referred codes from other assets integration and it may have mistakes.
I have written the UT and it passed