Commit def4ab3b authored by Tina's avatar Tina Committed by GitHub

feat: add time to sign analytics (#7140)

* add time to sign analytics

* move comment
parent bb6de905
...@@ -105,6 +105,7 @@ export function useUniswapXSwapCallback({ ...@@ -105,6 +105,7 @@ export function useUniswapXSwapCallback({
} }
} }
const beforeSign = Date.now()
const { signature, updatedOrder } = await signDutchOrder() const { signature, updatedOrder } = await signDutchOrder()
sendAnalyticsEvent(SwapEventName.SWAP_SIGNED, { sendAnalyticsEvent(SwapEventName.SWAP_SIGNED, {
...@@ -112,6 +113,7 @@ export function useUniswapXSwapCallback({ ...@@ -112,6 +113,7 @@ export function useUniswapXSwapCallback({
trade, trade,
allowedSlippage, allowedSlippage,
fiatValues, fiatValues,
timeToSignSinceRequestMs: Date.now() - beforeSign,
}), }),
...analyticsContext, ...analyticsContext,
}) })
......
...@@ -94,6 +94,7 @@ export function useUniversalRouterSwapCallback( ...@@ -94,6 +94,7 @@ export function useUniversalRouterSwapCallback(
} }
const gasLimit = calculateGasMargin(gasEstimate) const gasLimit = calculateGasMargin(gasEstimate)
setTraceData('gasLimit', gasLimit.toNumber()) setTraceData('gasLimit', gasLimit.toNumber())
const beforeSign = Date.now()
const response = await provider const response = await provider
.getSigner() .getSigner()
.sendTransaction({ ...tx, gasLimit }) .sendTransaction({ ...tx, gasLimit })
...@@ -101,6 +102,7 @@ export function useUniversalRouterSwapCallback( ...@@ -101,6 +102,7 @@ export function useUniversalRouterSwapCallback(
sendAnalyticsEvent(SwapEventName.SWAP_SIGNED, { sendAnalyticsEvent(SwapEventName.SWAP_SIGNED, {
...formatSwapSignedAnalyticsEventProperties({ ...formatSwapSignedAnalyticsEventProperties({
trade, trade,
timeToSignSinceRequestMs: Date.now() - beforeSign,
allowedSlippage: options.slippageTolerance, allowedSlippage: options.slippageTolerance,
fiatValues, fiatValues,
txHash: response.hash, txHash: response.hash,
......
...@@ -65,15 +65,19 @@ export const formatSwapSignedAnalyticsEventProperties = ({ ...@@ -65,15 +65,19 @@ export const formatSwapSignedAnalyticsEventProperties = ({
allowedSlippage, allowedSlippage,
fiatValues, fiatValues,
txHash, txHash,
timeToSignSinceRequestMs,
}: { }: {
trade: InterfaceTrade trade: InterfaceTrade
allowedSlippage: Percent allowedSlippage: Percent
fiatValues: { amountIn?: number; amountOut?: number } fiatValues: { amountIn?: number; amountOut?: number }
txHash?: string txHash?: string
timeToSignSinceRequestMs?: number
}) => ({ }) => ({
transaction_hash: txHash, transaction_hash: txHash,
token_in_amount_usd: fiatValues.amountIn, token_in_amount_usd: fiatValues.amountIn,
token_out_amount_usd: fiatValues.amountOut, token_out_amount_usd: fiatValues.amountOut,
// measures the amount of time the user took to sign the permit message or swap tx in their wallet
time_to_sign_since_request_ms: timeToSignSinceRequestMs,
...formatCommonPropertiesForTrade(trade, allowedSlippage), ...formatCommonPropertiesForTrade(trade, allowedSlippage),
}) })
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment