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

feat: add initial voltage analytics draft #1

Open
wants to merge 68 commits into
base: main
Choose a base branch
from

Conversation

diegofigs
Copy link
Contributor

@diegofigs diegofigs commented Mar 30, 2022

Done βœ…

  • Overview
  • Token(s)
  • Pair(s)
  • Pool(s)
  • Stake
  • Portfolio

In Progress 🟑

Not Started πŸ”΄

  • Ensure tx table breaks cache on subsequent fetches

Overview, Token(s), Pair(s), Pool(s) working as expected; still issues in Stake / Portfolio
@diegofigs diegofigs requested a review from mul53 March 30, 2022 11:09
src/components/Column/index.js Outdated Show resolved Hide resolved
src/components/Link.js Show resolved Hide resolved
src/components/PairTable.js Show resolved Hide resolved
src/components/PairTable.js Outdated Show resolved Hide resolved
src/components/PoolTable.js Outdated Show resolved Hide resolved
src/config/index.ts Outdated Show resolved Hide resolved
src/core/api/exchange.js Outdated Show resolved Hide resolved
@@ -65,31 +79,52 @@ function IndexPage() {

const [useUSD, setUseUSD] = useState(true);

const [liquidity, volume] = dayDatas
Copy link
Contributor

Choose a reason for hiding this comment

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

why are we replacing this?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Came up with issues when supplying the data as props to the chart: in essence I would get multiple data points per day which led to bizarre behavior in the tvl chart and multiple bars per day in the volume chart. This handles that by normalizing/aggregating data per day.

Copy link
Contributor

Choose a reason for hiding this comment

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

hmmm i think let's try to minimize the code changes

src/pages/tokens/[id].js Show resolved Hide resolved
src/pages/users/[id].js Show resolved Hide resolved
src/components/AccountSearch/index.js Outdated Show resolved Hide resolved
src/components/AccountSearch/index.js Outdated Show resolved Hide resolved
src/components/AppNavigation.js Outdated Show resolved Hide resolved
src/components/GlobalStats.js Outdated Show resolved Hide resolved
src/components/LPList/index.js Outdated Show resolved Hide resolved
src/components/SortableTable.js Outdated Show resolved Hide resolved
src/components/TokenIcon.js Outdated Show resolved Hide resolved
src/core/api/exchange.js Outdated Show resolved Hide resolved
src/pages/users/index.js Show resolved Hide resolved
src/pages/tokens/[id].js Show resolved Hide resolved
@diegofigs diegofigs requested a review from mul53 April 22, 2022 13: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