Skip to content

Commit a306dc9

Browse files
authored
chore(query-bar): add telemetry for default sort COMPASS-9763 (#7245)
* chore(query-bar): add telemetry for default sort COMPASS-9763 * chore(e2e): update assertion
1 parent 839891f commit a306dc9

File tree

3 files changed

+13
-0
lines changed

3 files changed

+13
-0
lines changed

packages/compass-crud/src/stores/crud-store.ts

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1596,13 +1596,19 @@ class CrudStoreImpl
15961596

15971597
if (onApply) {
15981598
const { isTimeSeries, isReadonly } = this.state;
1599+
const { defaultSortOrder } = this.preferences.getPreferences();
15991600
this.track(
16001601
'Query Executed',
16011602
{
16021603
has_projection:
16031604
!!query.project && Object.keys(query.project).length > 0,
16041605
has_skip: (query.skip ?? 0) > 0,
16051606
has_sort: !!query.sort && Object.keys(query.sort).length > 0,
1607+
default_sort: !defaultSortOrder
1608+
? 'none'
1609+
: /_id/.test(defaultSortOrder)
1610+
? '_id'
1611+
: 'natural',
16061612
has_limit: (query.limit ?? 0) > 0,
16071613
has_collation: !!query.collation,
16081614
changed_maxtimems: query.maxTimeMS !== DEFAULT_INITIAL_MAX_TIME_MS,

packages/compass-e2e-tests/tests/collection-documents-tab.test.ts

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -181,6 +181,7 @@ describe('Collection documents tab', function () {
181181
has_projection: false,
182182
has_skip: false,
183183
has_sort: false,
184+
default_sort: 'none',
184185
mode: 'list',
185186
used_regex: false,
186187
});
@@ -219,6 +220,7 @@ describe('Collection documents tab', function () {
219220
has_limit: true,
220221
has_projection: true,
221222
has_sort: true,
223+
default_sort: 'none',
222224
has_skip: true,
223225
mode: 'list',
224226
used_regex: false,

packages/compass-telemetry/src/telemetry-events.ts

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1786,6 +1786,11 @@ type QueryExecutedEvent = ConnectionScopedEvent<{
17861786
*/
17871787
has_sort: boolean;
17881788

1789+
/**
1790+
* Indicates which default sort was set in settings
1791+
*/
1792+
default_sort: 'natural' | '_id' | 'none';
1793+
17891794
/**
17901795
* Indicates whether the query includes a limit operation.
17911796
*/

0 commit comments

Comments
 (0)