Skip to content

Conversation

@zeusongit
Copy link
Contributor

Purpose

Cherrypicking
ADP tracking for in-canvas search control was being recorded incorrectly as it was also triggered right after startup.
Therefore, it is now moved to the function that handles visibility change for both the controls to maintain uniformity and accurate hits.

Reviewers

@QilongTang

zeusongit and others added 4 commits October 27, 2020 15:45
* Add list of nodes to be skipped

* Add test and comments
* add analytics coverage for node autocomplete

Open auto complete window
Select suggestion

* Update NodeAutoCompleteSearchControl.xaml.cs

* add tracking to in-canvas search node selection

* add track points for in canvas search open/select

Co-authored-by: Ashish Aggarwal <ashish.zeus17@gmail.com>
@zeusongit zeusongit requested a review from QilongTang November 5, 2020 16:12
@zeusongit zeusongit changed the title Cherrypich: Change analytics code block position for both controls, Nodeautocomplete and Incanvas search (#11232) Cherrypick: Change analytics code block position for both controls, Nodeautocomplete and Incanvas search (#11232) Nov 5, 2020
@zeusongit zeusongit changed the title Cherrypick: Change analytics code block position for both controls, Nodeautocomplete and Incanvas search (#11232) Cherry-pick: Change analytics code block position for both controls, Nodeautocomplete and Incanvas search (#11232) Nov 5, 2020
@QilongTang QilongTang merged commit 4c9cadd into DynamoDS:RC2.9.0_master Nov 5, 2020
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.

3 participants