Skip to content

Conversation

@SimSef
Copy link
Contributor

@SimSef SimSef commented Apr 15, 2025

…n method

@SimSef SimSef requested a review from galvanizze April 15, 2025 09:37
@SimSef SimSef self-assigned this Apr 15, 2025
Comment on lines 236 to 239
volume=str(stats.get("btcValue", 0)),
sales_count=str(stats.get("listed", 0)),
owners_count=str(stats.get("total", 0)),
market_cap=str(stats.get("btcValue", 0)),
Copy link
Member

Choose a reason for hiding this comment

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

q: do we process values in BTC? maybe USD would be better, but depends what we do with those data

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixed, with market cap being computed as floor_price * total_nfts and btcValue = volume

@SimSef SimSef force-pushed the 2149-blockapi-btc-nft-api-replacement-after-simplehash-shutdown branch from 512b163 to 6c0b63a Compare April 15, 2025 10:15
@SimSef SimSef merged commit 75c1c6d into master Apr 15, 2025
5 checks passed
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.

3 participants