fix(stats): improve stats app resiliency

[forbesus]

ref #330

[forbesus]

Hello @baktun14 would you check it?

[forbesus]

Hello @Redm4x @ygrishajev
I finished to fix
Would you check again?

[forbesus]

Hello @ygrishajev I just fixed
Would you check again?

[forbesus]

Hello @ygrishajev How about now?

[ygrishajev]

Thanks for all your changes @forbesus! This looks so much better!

I'm adding a couple of comments, that you may improve further if you like. Anyways lgtm.
Let's please make sure it's also good with @Redm4x

[ygrishajev]

what do you think about one last improvement here? Promise.all should speed up this endpoint quite a bit

const [{ now, compare }, chainStatsQuery, networkCapacity, networkCapacityStats, latestBlocks, latestTransactions] = await Promise.all([
    runOrLog(getDashboardData),
    runOrLog(getChainStats, {
      bondedTokens: undefined,
      totalSupply: undefined,
      communityPool: undefined,
      inflation: undefined,
      stakingAPR: undefined
    }),
    runOrLog(getNetworkCapacity),
    runOrLog(() => getProviderGraphData("count")),
    runOrLog(() => getBlocks(5)),
    runOrLog(() => getTransactions(5))
  ]);

  const chainStats = {
    ...chainStatsQuery,
    height: latestBlocks && latestBlocks.length > 0 ? latestBlocks[0].height : undefined,
    transactionCount: latestBlocks && latestBlocks.length > 0 ? latestBlocks[0].totalTransactionCount : undefined
  };

[ygrishajev]

oh, 1 thing before we can merge. Please squash the commits and use a scope for the resulting commits. e.g.

feat(stats): ...message

This should fail on pre-commit hook coz this scope is not added to our rules. Please feel free to add it.

@Redm4x does this scope make sense to you?

[forbesus]

All things are finished @Redm4x cc: @ygrishajev

[ygrishajev]

We follow rebase strategy, can you squash the merge commit too pls?

[forbesus]

How about now? @ygrishajev cc: @Redm4x

[ygrishajev]

Lgtm! Thanks for all your changes!