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

Feature/overview metrics #371

Merged
merged 7 commits into from
Jul 14, 2023
Merged

Feature/overview metrics #371

merged 7 commits into from
Jul 14, 2023

Conversation

ccali11
Copy link
Contributor

@ccali11 ccali11 commented Jul 12, 2023

@shanejearley @DemogorGod - Slacked you both. This isn't done because there are a few bugs and maybe some naming cleanup I can do, but this PR implements the three Dashboard/Overview metrics we have showing. I renamed them temporarily for clarity, but we can call them whatever we want. Currently, they are:

  • Total Balance Across Connected Wallets
  • Currently Staked
  • All Time Staking Rewards Earned

They update on refresh and whenever a user address is involved in a new Ethereum transaction as I'm listening to the blockchain for events. There's currently a bug in my contract listener that's preventing updates on StakeRebalanced events, but I will fix that tonight/tomorrow no problem. Ping me if you have any questions, but feel free to pull down and run to see close-to-complete functionality for entire Breakdown Chart + Staking component. :)

Copy link
Contributor

@DemogorGod DemogorGod left a comment

Choose a reason for hiding this comment

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

I like the way we are heading with the app, we'll get any work/bugs done in no time!

@ccali11 ccali11 merged commit 0036759 into develop Jul 14, 2023
@ccali11 ccali11 deleted the feature/overview-metrics branch July 14, 2023 04:12
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.

2 participants