Issue 588 ✨ feature invert all prices for stablecoin pairs#597
Hidden character warning
Issue 588 ✨ feature invert all prices for stablecoin pairs#597alchemist-bounty wants to merge 83 commits intodevelopmentfrom
Conversation
…_for_stablecoin_pairs
…_for_stablecoin_pairs
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
… issue-588-✨_Feature_Invert_all_prices_for_stablecoin_pairs
There was a problem hiding this comment.
isStable in most places should be called isInverted
stablecoins have isInverted set to true by default, otherwise false
All coins can toggle between inverted and non-inverted mode using the button @stephclarkga designed.
| // } | ||
|
|
||
| const handleChange = (e) => { | ||
| if (isBuyOrder && price === '0') { |
There was a problem hiding this comment.
Does this have the fixes we recently put into the place order form?
| fetchCurrentAssetPrices | ||
| } from 'services/algoexplorer' | ||
|
|
||
| import { routeQueryError } from './useAlgodex' |
There was a problem hiding this comment.
It is not necessary to add this file since I ported all the hooks into /hooks. Check /hooks/http/useAlgoExplorer.js
| import ServiceError from '@/components/ServiceError' | ||
| import Spinner from '@/components/Spinner' | ||
| import withQuery from '@/hooks/withQuery' | ||
|
|
There was a problem hiding this comment.
See above comment. We might already have this file in our repo?
| } | ||
| const { | ||
| data: { asset } | ||
| } = await axios.get(`${EXPLORER_INDEXER_API}/v2/assets/${id}?include-all=true`) |
There was a problem hiding this comment.
Shouldn't this use React query instead of axios?
3 participants
ℹ Overview
📝 Related Issues
🔐 Acceptance:
yarn testpasses