Skip to content

Commit 1796808

Browse files
committed
remove empty localization strings
1 parent 626028f commit 1796808

File tree

5 files changed

+6
-24
lines changed

5 files changed

+6
-24
lines changed

src/messageTypes.ts

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -193,7 +193,6 @@ export type LocalizedMessages = {
193193
integrationsPostgresPasswordLabel: string;
194194
integrationsPostgresPasswordPlaceholder: string;
195195
integrationsPostgresSslLabel: string;
196-
integrationsPostgresUnnamedIntegration: string;
197196
// BigQuery form strings
198197
integrationsBigQueryNameLabel: string;
199198
integrationsBigQueryNamePlaceholder: string;
@@ -202,8 +201,6 @@ export type LocalizedMessages = {
202201
integrationsBigQueryCredentialsLabel: string;
203202
integrationsBigQueryCredentialsPlaceholder: string;
204203
integrationsBigQueryCredentialsRequired: string;
205-
integrationsBigQueryInvalidJson: string;
206-
integrationsBigQueryUnnamedIntegration: string;
207204
// Common form strings
208205
integrationsRequiredField: string;
209206
integrationsOptionalField: string;

src/notebooks/deepnote/integrations/integrationWebview.ts

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -130,16 +130,13 @@ export class IntegrationWebviewProvider implements IIntegrationWebviewProvider {
130130
integrationsPostgresPasswordLabel: localize.Integrations.postgresPasswordLabel,
131131
integrationsPostgresPasswordPlaceholder: localize.Integrations.postgresPasswordPlaceholder,
132132
integrationsPostgresSslLabel: localize.Integrations.postgresSslLabel,
133-
integrationsPostgresUnnamedIntegration: '', // Will be formatted on the client side
134133
integrationsBigQueryNameLabel: localize.Integrations.bigQueryNameLabel,
135134
integrationsBigQueryNamePlaceholder: localize.Integrations.bigQueryNamePlaceholder,
136135
integrationsBigQueryProjectIdLabel: localize.Integrations.bigQueryProjectIdLabel,
137136
integrationsBigQueryProjectIdPlaceholder: localize.Integrations.bigQueryProjectIdPlaceholder,
138137
integrationsBigQueryCredentialsLabel: localize.Integrations.bigQueryCredentialsLabel,
139138
integrationsBigQueryCredentialsPlaceholder: localize.Integrations.bigQueryCredentialsPlaceholder,
140-
integrationsBigQueryCredentialsRequired: localize.Integrations.bigQueryCredentialsRequired,
141-
integrationsBigQueryInvalidJson: '', // Will be formatted on the client side
142-
integrationsBigQueryUnnamedIntegration: '' // Will be formatted on the client side
139+
integrationsBigQueryCredentialsRequired: localize.Integrations.bigQueryCredentialsRequired
143140
};
144141

145142
await this.currentPanel.webview.postMessage({

src/platform/webviews/webviewHost.ts

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -293,16 +293,13 @@ export abstract class WebviewHost<IMapping> implements IDisposable {
293293
integrationsPostgresPasswordLabel: '',
294294
integrationsPostgresPasswordPlaceholder: '',
295295
integrationsPostgresSslLabel: '',
296-
integrationsPostgresUnnamedIntegration: '',
297296
integrationsBigQueryNameLabel: '',
298297
integrationsBigQueryNamePlaceholder: '',
299298
integrationsBigQueryProjectIdLabel: '',
300299
integrationsBigQueryProjectIdPlaceholder: '',
301300
integrationsBigQueryCredentialsLabel: '',
302301
integrationsBigQueryCredentialsPlaceholder: '',
303302
integrationsBigQueryCredentialsRequired: '',
304-
integrationsBigQueryInvalidJson: '',
305-
integrationsBigQueryUnnamedIntegration: '',
306303
integrationsRequiredField: '',
307304
integrationsOptionalField: ''
308305
};

src/webviews/webview-side/integrations/BigQueryForm.tsx

Lines changed: 3 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
import * as React from 'react';
2-
import { getLocString } from '../react-common/locReactSide';
2+
import { format, getLocString } from '../react-common/locReactSide';
33
import { BigQueryIntegrationConfig } from './types';
44

55
export interface IBigQueryFormProps {
@@ -37,10 +37,7 @@ export const BigQueryForm: React.FC<IBigQueryFormProps> = ({ integrationId, exis
3737
return true;
3838
} catch (error) {
3939
const errorMessage = error instanceof Error ? error.message : 'Invalid JSON format';
40-
const invalidJsonMsg = getLocString('integrationsBigQueryInvalidJson', 'Invalid JSON: {0}').replace(
41-
'{0}',
42-
errorMessage
43-
);
40+
const invalidJsonMsg = format('Invalid JSON: {0}', errorMessage);
4441
setCredentialsError(invalidJsonMsg);
4542
return false;
4643
}
@@ -60,10 +57,7 @@ export const BigQueryForm: React.FC<IBigQueryFormProps> = ({ integrationId, exis
6057
return;
6158
}
6259

63-
const unnamedIntegration = getLocString(
64-
'integrationsBigQueryUnnamedIntegration',
65-
'Unnamed BigQuery Integration ({0})'
66-
).replace('{0}', integrationId);
60+
const unnamedIntegration = format('Unnamed BigQuery Integration ({0})', integrationId);
6761

6862
const config: BigQueryIntegrationConfig = {
6963
id: integrationId,

src/webviews/webview-side/integrations/PostgresForm.tsx

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
import * as React from 'react';
2-
import { getLocString } from '../react-common/locReactSide';
2+
import { format, getLocString } from '../react-common/locReactSide';
33
import { PostgresIntegrationConfig } from './types';
44

55
export interface IPostgresFormProps {
@@ -34,10 +34,7 @@ export const PostgresForm: React.FC<IPostgresFormProps> = ({ integrationId, exis
3434
const handleSubmit = (e: React.FormEvent) => {
3535
e.preventDefault();
3636

37-
const unnamedIntegration = getLocString(
38-
'integrationsPostgresUnnamedIntegration',
39-
'Unnamed PostgreSQL Integration ({0})'
40-
).replace('{0}', integrationId);
37+
const unnamedIntegration = format('Unnamed PostgreSQL Integration ({0})', integrationId);
4138

4239
const config: PostgresIntegrationConfig = {
4340
id: integrationId,

0 commit comments

Comments
 (0)