Skip to content

fix(perps): update batch close analytics#6515

Merged
SeniorZhai merged 2 commits into
feat/tradingview-candle-chartfrom
fix/perps-batch-close-analytics
Jul 22, 2026
Merged

fix(perps): update batch close analytics#6515
SeniorZhai merged 2 commits into
feat/tradingview-candle-chartfrom
fix/perps-batch-close-analytics

Conversation

@SeniorZhai

Copy link
Copy Markdown
Member

No description provided.

Copilot AI review requested due to automatic review settings July 22, 2026 09:42

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates batch perpetual-position closing to use the unified close analytics funnel.

Changes:

  • Records batch closes with the multiple close type.
  • Aligns batch confirm, cancel, and completion events with single-position analytics.
  • Removes the obsolete preview event.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
PerpsBatchCloseBottomSheetDialogFragment.kt Uses unified close analytics events.
AllPositionsFragment.kt Records batch-close starts as multiple.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@SeniorZhai
SeniorZhai changed the base branch from master to feat/tradingview-candle-chart July 22, 2026 09:44
@SeniorZhai
SeniorZhai merged commit 6671a22 into feat/tradingview-candle-chart Jul 22, 2026
@SeniorZhai
SeniorZhai deleted the fix/perps-batch-close-analytics branch July 22, 2026 09:45
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