Skip to content

feat: track expand/collapse scope/variable usage#64

Open
barinali wants to merge 1 commit intointroduce-trackingfrom
collapse-expand-events
Open

feat: track expand/collapse scope/variable usage#64
barinali wants to merge 1 commit intointroduce-trackingfrom
collapse-expand-events

Conversation

@barinali
Copy link
Contributor

Proposed Changes

This pull request aims to introduce a tracking event for the collapse/expand scope/variable action.

Event Name Action
collapseScope User collapses a scope section
expandScope User expands a scope section
collapseVariable User collapses a variable row
expandVariable User expands a variable row

Checklist

Ensure you provide everything we need to review your contribution:

  • Contribution meets our definition of done
  • Pull request establishes context
    • Link to related issue(s), i.e. Closes {LINK_TO_ISSUE} or Related to {LINK_TO_ISSUE}
    • Brief textual description of the changes
    • Screenshots or short videos showing UI/UX changes
    • Steps to try out, i.e. using the @bpmn-io/sr tool

@barinali barinali self-assigned this Mar 23, 2026
Copilot AI review requested due to automatic review settings March 23, 2026 12:51
@bpmn-io-tasks bpmn-io-tasks bot added the needs review Review pending label Mar 23, 2026
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds analytics tracking for user expand/collapse interactions in the variable outline UI, leveraging the existing useTracking() hook to emit namespaced events.

Changes:

  • Emit expandScope / collapseScope tracking events when toggling a scope via useScopeExpand.
  • Emit expandVariable / collapseVariable tracking events when toggling individual variable rows in Scope.
  • Add unit tests covering the new tracking behavior and document the change in the changelog.

Reviewed changes

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

Show a summary per file
File Description
lib/hooks/useScopeExpand.js Adds scope expand/collapse tracking on toggle via useTracking()
lib/hooks/test/useScopeExpand.spec.jsx New tests verifying expandScope / collapseScope events are fired
lib/components/Scope/Scope.jsx Adds variable row expand/collapse tracking on per-row toggle
lib/components/Scope/test/Scope.spec.jsx New tests verifying expandVariable / collapseVariable events are fired
CHANGELOG.md Notes the new tracking feature in Unreleased section

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs review Review pending

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants