Skip to content

Commit 94843c9

Browse files
committed
show "unknown integration" on block when not connected
1 parent 1faed39 commit 94843c9

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

src/notebooks/deepnote/sqlCellStatusBarProvider.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -97,7 +97,7 @@ export class SqlCellStatusBarProvider implements NotebookCellStatusBarItemProvid
9797

9898
// Get integration configuration to display the name
9999
const config = await this.integrationStorage.getIntegrationConfig(projectId, integrationId);
100-
const displayName = config?.name || integrationId;
100+
const displayName = config?.name || l10n.t('Unknown integration (configure)');
101101

102102
// Create a status bar item that opens the integration management UI
103103
return {

src/notebooks/deepnote/sqlCellStatusBarProvider.unit.test.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -89,7 +89,7 @@ suite('SqlCellStatusBarProvider', () => {
8989
assert.deepStrictEqual((result as any).command.arguments, [integrationId]);
9090
});
9191

92-
test('uses integration ID as display name when config not found', async () => {
92+
test('shows "Unknown integration (configure)" when config not found', async () => {
9393
const integrationId = 'postgres-123';
9494
const cell = createMockCell(
9595
'sql',
@@ -106,7 +106,7 @@ suite('SqlCellStatusBarProvider', () => {
106106
const result = await provider.provideCellStatusBarItems(cell, cancellationToken);
107107

108108
assert.isDefined(result);
109-
assert.strictEqual((result as any).text, '$(database) postgres-123');
109+
assert.strictEqual((result as any).text, '$(database) Unknown integration (configure)');
110110
});
111111

112112
test('returns undefined when notebook has no project ID', async () => {

0 commit comments

Comments
 (0)