Skip to content

Commit 8de2ebf

Browse files
Merge remote-tracking branch 'origin/main' into beta-releases
2 parents 61159ed + 13aac25 commit 8de2ebf

File tree

13 files changed

+214
-333
lines changed

13 files changed

+214
-333
lines changed

THIRD-PARTY-NOTICES.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
The following third-party software is used by and included in **Mongodb Compass**.
2-
This document was automatically generated on Thu Jul 18 2024.
2+
This document was automatically generated on Sun Jul 21 2024.
33

44
## List of dependencies
55

package-lock.json

Lines changed: 0 additions & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

packages/compass-connections/src/components/legacy-connections.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -199,7 +199,7 @@ function Connections({
199199
<ConnectionForm
200200
onConnectClicked={onConnectClick}
201201
key={activeConnectionId}
202-
onSaveConnectionClicked={saveConnection}
202+
onSaveClicked={saveConnection}
203203
initialConnectionInfo={activeConnectionInfo}
204204
connectionErrorMessage={connectionErrorMessage}
205205
preferences={preferences}

packages/compass-sidebar/src/components/multiple-connections/sidebar.tsx

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -367,16 +367,17 @@ export function MultipleConnectionSidebar({
367367
[]
368368
);
369369

370-
const onNewConnectionConnect = useCallback(
370+
const onSaveAndConnectClicked = useCallback(
371371
(connectionInfo: ConnectionInfo) => {
372+
void saveConnection(connectionInfo);
372373
void _onConnect(connectionInfo).then(() => {
373374
setIsConnectionFormOpen(false);
374375
});
375376
},
376-
[_onConnect]
377+
[saveConnection, _onConnect]
377378
);
378379

379-
const onSaveNewConnection = useCallback(
380+
const onSaveConnectionClicked = useCallback(
380381
async (connectionInfo: ConnectionInfo) => {
381382
await saveConnection(connectionInfo);
382383
setIsConnectionFormOpen(false);
@@ -501,9 +502,9 @@ export function MultipleConnectionSidebar({
501502
isOpen={isConnectionFormOpen}
502503
setOpen={onNewConnectionToggle}
503504
onCancel={onNewConnectionClose}
504-
onConnectClicked={onNewConnectionConnect}
505+
onSaveAndConnectClicked={onSaveAndConnectClicked}
505506
key={activeConnectionId}
506-
onSaveConnectionClicked={onSaveNewConnection}
507+
onSaveClicked={onSaveConnectionClicked}
507508
initialConnectionInfo={activeConnectionInfo}
508509
connectionErrorMessage={connectionErrorMessage}
509510
preferences={formPreferences}

packages/compass/src/app/utils/telemetry.spec.ts

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -40,6 +40,7 @@ const connectionInfo: ConnectionInfo = {
4040
connectionOptions: {
4141
connectionString: 'mongodb://localhost:27017',
4242
},
43+
savedConnectionType: 'recent',
4344
};
4445

4546
describe('connection tracking', function () {
@@ -67,6 +68,7 @@ describe('connection tracking', function () {
6768
...connectionInfo,
6869
favorite: { name: 'example' },
6970
lastUsed: undefined,
71+
savedConnectionType: 'favorite',
7072
},
7173
logger,
7274
track
@@ -119,6 +121,7 @@ describe('connection tracking', function () {
119121
{
120122
...connectionInfo,
121123
favorite: { name: 'example' },
124+
savedConnectionType: 'favorite',
122125
lastUsed: new Date(),
123126
},
124127
logger,

packages/compass/src/app/utils/telemetry.ts

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -156,10 +156,11 @@ export function trackConnectionAttemptEvent(
156156
track: TrackFunction
157157
): void {
158158
try {
159-
const { favorite, lastUsed } = connectionInfo;
159+
const { lastUsed, savedConnectionType } = connectionInfo;
160+
const isFavorite = savedConnectionType === 'favorite';
160161
const trackEvent = {
161-
is_favorite: Boolean(favorite),
162-
is_recent: Boolean(lastUsed && !favorite),
162+
is_favorite: isFavorite,
163+
is_recent: Boolean(lastUsed && !isFavorite),
163164
is_new: !lastUsed,
164165
};
165166
track('Connection Attempt', trackEvent, connectionInfo);

0 commit comments

Comments
 (0)