-
Notifications
You must be signed in to change notification settings - Fork 984
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
[#10041] Request token price after adding #10043
Conversation
Pull Request Checklist
|
Jenkins BuildsClick to see older builds (3)
|
"from" from | ||
"to" to | ||
"mainnet?" mainnet? | ||
"chaos-mode?" chaos-mode?) |
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.
didn't we remove chaos-mode?
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.
I have no idea, it is at least in this function
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.
@yenda as far as I see it is still in the project
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.
i just disabled it, not removed, but dunno if we want to remove all related code completely
@rasom related bug, when we remove token we don't recalculate total value in fiat, probably could be done in this PR ? |
9c91f8b
to
abeee10
Compare
abeee10
to
965dce0
Compare
fix #10041
status: ready