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

add today's token balance from the previous value #1756

Merged
merged 7 commits into from
Apr 16, 2019

Conversation

ayrat555
Copy link
Contributor

fixes #1754

Changelog

  • add today's token balance from the previous value

@ghost ghost assigned ayrat555 Apr 15, 2019
@ghost ghost added the in progress label Apr 15, 2019
@coveralls
Copy link

coveralls commented Apr 15, 2019

Pull Request Test Coverage Report for Build 183d3d2f-1735-414a-8c3a-8fd0795c44d3

  • 8 of 8 (100.0%) changed or added relevant lines in 1 file are covered.
  • 4 unchanged lines in 1 file lost coverage.
  • Overall coverage decreased (-0.06%) to 83.008%

Files with Coverage Reduction New Missed Lines %
apps/indexer/lib/indexer/block/catchup/fetcher.ex 4 68.0%
Totals Coverage Status
Change from base Build c483421c-4e5d-4afd-bcdf-045ab6d4c061: -0.06%
Covered Lines: 4416
Relevant Lines: 5320

💛 - Coveralls

assert balances == [
%{date: "2019-04-15", value: Decimal.new("1E-15")},
%{date: "2019-04-14", value: Decimal.new("1E-15")}
]
Copy link
Contributor

Choose a reason for hiding this comment

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

Won't this fail tomorrow?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@goodsoft I removed hardcoded values

@ayrat555 ayrat555 requested a review from goodsoft April 16, 2019 08:54
@ayrat555 ayrat555 merged commit 6a96128 into master Apr 16, 2019
@ghost ghost removed the in progress label Apr 16, 2019
@ayrat555 ayrat555 deleted the ab-interpolate-token-balance-to-current-data branch April 16, 2019 09:16
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.

interpolate balance chart on x axis
4 participants