Skip to content

Commit

Permalink
Merge pull request #483 from reservoirprotocol/ted/fix-slippage-for-s…
Browse files Browse the repository at this point in the history
…ame-chain-swaps

Fix slippage displayed for same-chain-swaps
  • Loading branch information
ted-palmer authored Feb 21, 2025
2 parents 67a7f6d + 5be1e1d commit 9b08e07
Show file tree
Hide file tree
Showing 3 changed files with 13 additions and 9 deletions.
5 changes: 5 additions & 0 deletions .changeset/neat-tigers-decide.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
'@reservoir0x/relay-kit-ui': patch
---

Fix slippage displayed for same-chain-swaps
Original file line number Diff line number Diff line change
Expand Up @@ -233,11 +233,10 @@ export const ReviewQuoteStep: FC<ReviewQuoteProps> = ({
)
})

const slippage =
quote?.details?.slippageTolerance?.destination?.percent ??
quote?.details?.slippageTolerance?.origin?.percent ??
'0'

const isSameChain = toToken?.chainId === fromToken?.chainId
const slippage = isSameChain
? quote?.details?.slippageTolerance?.origin?.percent ?? '0'
: quote?.details?.slippageTolerance?.destination?.percent ?? '0'
const slippageRating = getSlippageRating(slippage)
const slippageRatingColor = ratingToColor[slippageRating]

Expand Down
8 changes: 4 additions & 4 deletions packages/ui/src/components/widgets/FeeBreakdown.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -71,10 +71,10 @@ const FeeBreakdown: FC<Props> = ({
const [rateMode, setRateMode] = useState<'input' | 'output'>('input')

const isHighPriceImpact = Number(price?.details?.totalImpact?.percent) < -3.5
const slippage =
price?.details?.slippageTolerance?.destination?.percent ??
price?.details?.slippageTolerance?.origin?.percent ??
'0'
const isSameChain = toToken?.chainId === fromToken?.chainId
const slippage = isSameChain
? price?.details?.slippageTolerance?.origin?.percent ?? '0'
: price?.details?.slippageTolerance?.destination?.percent ?? '0'

const slippageRating = getSlippageRating(slippage)
const slippageRatingColor = ratingToColor[slippageRating]
Expand Down

0 comments on commit 9b08e07

Please sign in to comment.