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

Transactions history chart #3065

Merged
merged 55 commits into from
Apr 9, 2020
Merged

Transactions history chart #3065

merged 55 commits into from
Apr 9, 2020

Conversation

vbaranov
Copy link
Member

@vbaranov vbaranov commented Apr 6, 2020

Origin in #2480
Closes #811
Closes #2985

Motivation

Add Transactions history chart

A new set of environment variables:

Variable Required Description Default Version Need recompile
ENABLE_TX_STATS Disables or enables txs per day stats gathering false v3.1.2+
DISABLE_EXCHANGE_RATES Disables or enables fetching of coin price from Coingecko API false v3.1.2+
SHOW_PRICE_CHART Disables or enables price and market cap of coin charts on the main page false v3.1.2+
SHOW_TXS_CHART Disables or enables txs count per day chart on the main page false v3.1.2+
HISTORY_FETCH_INTERVAL Interval in minutes how often to request count of txs per current day in order to display txs count per day chart on the main page 60 v3.1.2+
TXS_HISTORIAN_INIT_LAG The initial delay in minutes in txs count history fetching in order to display txs count per day history chart on the main page 0 v3.1.2+
TXS_STATS_DAYS_TO_COMPILE_AT_INIT Number of days for fetching of history of txs count per day in order to display it in txs count per day history chart on the main page 365 v3.1.2+

Related PR to update docs blockscout/docs#16

Checklist for your Pull Request (PR)

@vbaranov vbaranov added the wip label Apr 6, 2020
@vbaranov vbaranov force-pushed the vb-transaction-history branch 2 times, most recently from 1873ed6 to e48a590 Compare April 6, 2020 09:29
@coveralls
Copy link

coveralls commented Apr 6, 2020

Pull Request Test Coverage Report for Build 894abfaa-b76a-467d-9e27-78468e816017

  • 70 of 80 (87.5%) changed or added relevant lines in 8 files are covered.
  • 1 unchanged line in 1 file lost coverage.
  • Overall coverage increased (+0.1%) to 74.943%

Changes Missing Coverage Covered Lines Changed/Added Lines %
apps/block_scout_web/lib/block_scout_web/controllers/chain_controller.ex 10 11 90.91%
apps/explorer/lib/explorer/chain/transaction/history/transaction_stats.ex 2 3 66.67%
apps/explorer/lib/explorer/history/process.ex 20 22 90.91%
apps/block_scout_web/lib/block_scout_web/notifier.ex 0 6 0.0%
Files with Coverage Reduction New Missed Lines %
apps/block_scout_web/lib/block_scout_web/controllers/chain/market_history_chart_controller.ex 1 78.57%
Totals Coverage Status
Change from base Build 845910f5-cf6e-4dd9-9ccc-5d9bc05bffa0: 0.1%
Covered Lines: 5611
Relevant Lines: 7487

💛 - Coveralls

@vbaranov vbaranov force-pushed the vb-transaction-history branch 8 times, most recently from c48403f to 039e4a4 Compare April 6, 2020 18:46
@vbaranov vbaranov added ready for review This PR is ready for reviews. and removed wip labels Apr 8, 2020
@vbaranov vbaranov mentioned this pull request Apr 9, 2020
@vbaranov vbaranov merged commit 29e2978 into master Apr 9, 2020
@vbaranov vbaranov deleted the vb-transaction-history branch April 9, 2020 15:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready for review This PR is ready for reviews.
Projects
None yet
3 participants