Skip to content
This repository has been archived by the owner on Apr 20, 2022. It is now read-only.

Improve market cap #336

Merged
merged 2 commits into from
Aug 14, 2021
Merged

Improve market cap #336

merged 2 commits into from
Aug 14, 2021

Conversation

rootulp
Copy link
Contributor

@rootulp rootulp commented Aug 9, 2021

Fixes #330

If a market cap is provided, display it.
If total supply is provided, approximate market cap as supply * price. This may be useful for future product launches where CoinGecko doesn't provide market cap on day 1.
If market cap and supply aren't provided, display "--"

Screenshots for DPI

Scenario Screenshot
If market cap is provided Screen Shot 2021-08-09 at 6 10 05 PM
If total supply is provided Screen Shot 2021-08-09 at 6 09 42 PM
Neither are provided Screen Shot 2021-08-09 at 6 09 22 PM

Tagging @0xModene

Fixes IndexCoop#330

If a market cap is provided, display it.
If total supply is provided, approximate market cap as supply * price.
If market cap and supply aren't provided, display "--"
Copy link
Contributor

@controtie controtie left a comment

Choose a reason for hiding this comment

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

LGTM

@controtie controtie merged commit 52c08e3 into IndexCoop:master Aug 14, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

BED Page and other Token Stats bugs
2 participants