feat: Top accounts - Githubissues

aeternity / aescan

Block Explorer and Analytics Platform

ISC License

6 stars 3 forks source link

feat: Top accounts #634

Closed janmichek closed 7 months ago

janmichek commented 10 months ago

Description

resolves #616

The endpoint responds with top 200 accounts. Cannot be limited to 100. I think we can proceed with that
There is no entrypoint for the feature yet. So the only way to check the feature is to go to /top-accounts URL. The entripoint is a part of new menu here https://github.com/aeternity/aescan/pull/627

Demo

Checklist:

[x] I have read and followed the Contributing Guide
[X] add top accounts page

github-actions[bot] commented 10 months ago

Deployed to https://pr-634-aescan.stg.service.aepps.com

janmichek commented 10 months ago

According to Top Accounts Info: The top accounts are the accounts with the highest balance of AE coins. But I can see in the list that many accounts have 0 balance now and the last Tx was 10 months ago or 1-2 year ago. Is this correct order?

Good catch, was not checking it. I reported this to MDW https://github.com/aeternity/ae_mdw/issues/1661 and putting this issue On Hold until resolved

janmichek commented 10 months ago

According to https://github.com/aeternity/aescan/issues/616 description A new table should contain the following columns:

Balance added

michele-franchi commented 7 months ago

Closing this until the issue is not fixed on the mdw.