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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Display forged blocks instead of forged arks in delegate monitor list... #210

Merged
merged 1 commit into from May 29, 2018

Conversation

Projects
None yet
2 participants
@air1one
Copy link
Contributor

air1one commented May 29, 2018

... and avoid making 51 requests at every timer refresh.

I started looking at #195 and although this does not concern activeDelegates method directly, it could help a lot reducing server requests.

To me, displaying forged blocks instead of arks makes sense, and we can still see forged arks in the delegate details.
Now tell me what you think 馃槂

@faustbrian

This comment has been minimized.

Copy link
Contributor

faustbrian commented May 29, 2018

Agree, this should reduce the load and if needed we could just do producedBlocks * 2 later on for a rough estimate of forged ARK, excluding forged fees.

@faustbrian faustbrian merged commit bd37e55 into ArkEcosystem:master May 29, 2018

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment