Merged
Conversation
Require approval from @Danswar or @TaprootFreak for all PRs to protected branches. Co-authored-by: Danswar <48102227+Danswar@users.noreply.github.com>
* fix: use real Coingecko prices for testnet tokens
- Add TESTNET_COINGECKO_MAPPING to map token symbols to Coingecko IDs
- WCBTC now fetches real Bitcoin price (~$100k) instead of fake $1
- Fixes incorrect collateralization display (was 0.01%, now ~200%)
- Fallback to $1 for stablecoins and unknown tokens
* fix: add null safety and improve error logging
* fix: return only USD from Coingecko, let updatePrices() calculate JUSD price
The 'eur' field is actually the price in JUSD (protocol stablecoin), not Euro.
By only returning USD, the correct JUSD price calculation in updatePrices()
is triggered: eur = usd / protocolStablecoinPrice
* chore: remove unused randRef variable
* Remove EUR/JUSD pricing complexity - use USD only
Since 1 JUSD = 1 USD (dollar stablecoin), there's no need for
currency conversion. This simplifies the pricing system:
- Remove /prices/eur endpoint from controller
- Remove getEuroPrice(), fetchEuroPrice() from service
- Remove euroPrice state variable
- Simplify PriceQueryCurrencies to just { usd?, btc? }
- Remove jusd calculations from ecosystem collateral stats
The API now works exclusively with USD, which is equivalent
to JUSD at 1:1 ratio.
* chore: use bitcoin ID for WBTC, remove dead code
* ci: add PR checks workflow for build and lint
* chore: rebrand d-EURO to JuiceDollar in telegram messages
* chore: update LICENSE copyright
* perf: skip Coingecko for JUSD - stablecoin is always $1
* fix: handle missing Coingecko price data gracefully
- Add explicit check for undefined price data on mainnet
- Use price?.usd checks instead of price === null
- Log warning when no price data available
- Prevents NaN calculations from undefined prices
* fix: prevent TypeError when poolSharesPrice is null
Danswar
previously approved these changes
Jan 12, 2026
Deploy to both jdta and jdma container apps in DEV and PRD workflows.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.