mirror of
https://github.com/kremalicious/metamask-extension.git
synced 2024-12-21 00:43:21 +01:00
bb3a0423d4
This fixes a PropType error when using non-custom slippage, and it fixes a type inconsistency when custom slippage is used. Previously, `slippage` was being converted explicitly to a `Number` as it was passed into `BuildQuote`, but not as it was passed into `AwaitingSwap`. Also the PropType was set as `string`, despite the fact that it's a number in most cases, and is used for math. The PropType has been changed to `number`, and the selective casting to `Number` has been removed. Instead, the `maxSlippage` value is cast to a `Number` as it's being selected, so that the type is always consistent.
18 lines
714 B
JavaScript
18 lines
714 B
JavaScript
import PropTypes from 'prop-types'
|
|
|
|
export const QUOTE_DATA_ROWS_PROPTYPES_SHAPE = PropTypes.shape({
|
|
aggId: PropTypes.string.isRequired,
|
|
amountReceiving: PropTypes.string.isRequired,
|
|
destinationTokenDecimals: PropTypes.number.isRequired,
|
|
destinationTokenSymbol: PropTypes.string.isRequired,
|
|
destinationTokenValue: PropTypes.string.isRequired,
|
|
isBestQuote: PropTypes.bool,
|
|
networkFees: PropTypes.string.isRequired,
|
|
quoteSource: PropTypes.string.isRequired,
|
|
rawNetworkFees: PropTypes.string.isRequired,
|
|
slippage: PropTypes.number.isRequired,
|
|
sourceTokenDecimals: PropTypes.number.isRequired,
|
|
sourceTokenSymbol: PropTypes.string.isRequired,
|
|
sourceTokenValue: PropTypes.string.isRequired,
|
|
})
|