Skip to content

fix: Correct swap analytics duration measurement#1849

Open
kws1207 wants to merge 2 commits intodevelopfrom
fix/swap-quote-duration-measurement
Open

fix: Correct swap analytics duration measurement#1849
kws1207 wants to merge 2 commits intodevelopfrom
fix/swap-quote-duration-measurement

Conversation

@kws1207
Copy link
Collaborator

@kws1207 kws1207 commented Feb 23, 2026

  • total_duration_msswap_quote_received 시점부터만 측정되어 quote 요청 구간(swap_quote_requestedswap_quote_received)이 누락되는 버그 수정. durationRef 초기화 시 requestStartedAtRef.current를 first로 사용
  • fetching 중 파라미터 변경 시 queryRouteForLog 객체가 교체되어도 isFetching이 true → true로 effect가 재실행되지 않아 requestStartedAtRef가 갱신되지 않는 버그 수정. queryRouteForLog 객체 참조 변경도 감지하도록 dependency 추가

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@kws1207 kws1207 self-assigned this Feb 23, 2026
@kws1207 kws1207 requested a review from a team as a code owner February 23, 2026 11:02
@vercel
Copy link

vercel bot commented Feb 23, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
keplr-wallet-extension Ready Ready Preview, Comment Mar 3, 2026 7:44am

Request Review

…acking

Move `requestStartedAtRef.current = undefined` after the logEvent call
so that logEvent can use the value when initializing durationRef.first
for total_duration_ms calculation.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants