Skip to content

Commit 4b38b91

Browse files
committed
fix showing existing config
1 parent 7bd3712 commit 4b38b91

File tree

3 files changed

+25
-2
lines changed

3 files changed

+25
-2
lines changed

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

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,15 @@ export const BigQueryForm: React.FC<IBigQueryFormProps> = ({ integrationId, exis
1616
const [projectId, setProjectId] = React.useState(existingConfig?.projectId || '');
1717
const [credentials, setCredentials] = React.useState(existingConfig?.credentials || '');
1818

19+
// Update form fields when existingConfig changes
20+
React.useEffect(() => {
21+
if (existingConfig) {
22+
setName(existingConfig.name || '');
23+
setProjectId(existingConfig.projectId || '');
24+
setCredentials(existingConfig.credentials || '');
25+
}
26+
}, [existingConfig]);
27+
1928
const handleSubmit = (e: React.FormEvent) => {
2029
e.preventDefault();
2130

@@ -82,4 +91,3 @@ export const BigQueryForm: React.FC<IBigQueryFormProps> = ({ integrationId, exis
8291
</form>
8392
);
8493
};
85-

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

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -65,6 +65,7 @@ export const IntegrationPanel: React.FC<IIntegrationPanelProps> = ({ baseTheme,
6565
const handleSave = (config: IntegrationConfig) => {
6666
vscodeApi.postMessage({
6767
type: 'save',
68+
integrationId: config.id,
6869
config
6970
});
7071
setSelectedIntegrationId(null);

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

Lines changed: 15 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,8 @@ export interface IPostgresFormProps {
1212
}
1313

1414
export const PostgresForm: React.FC<IPostgresFormProps> = ({ integrationId, existingConfig, onSave, onCancel }) => {
15+
console.log('[PostgresForm] Rendering with existingConfig:', existingConfig);
16+
1517
const [name, setName] = React.useState(existingConfig?.name || '');
1618
const [host, setHost] = React.useState(existingConfig?.host || '');
1719
const [port, setPort] = React.useState(existingConfig?.port?.toString() || '5432');
@@ -20,6 +22,19 @@ export const PostgresForm: React.FC<IPostgresFormProps> = ({ integrationId, exis
2022
const [password, setPassword] = React.useState(existingConfig?.password || '');
2123
const [ssl, setSsl] = React.useState(existingConfig?.ssl || false);
2224

25+
// Update form fields when existingConfig changes
26+
React.useEffect(() => {
27+
if (existingConfig) {
28+
setName(existingConfig.name || '');
29+
setHost(existingConfig.host || '');
30+
setPort(existingConfig.port?.toString() || '5432');
31+
setDatabase(existingConfig.database || '');
32+
setUsername(existingConfig.username || '');
33+
setPassword(existingConfig.password || '');
34+
setSsl(existingConfig.ssl || false);
35+
}
36+
}, [existingConfig]);
37+
2338
const handleSubmit = (e: React.FormEvent) => {
2439
e.preventDefault();
2540

@@ -139,4 +154,3 @@ export const PostgresForm: React.FC<IPostgresFormProps> = ({ integrationId, exis
139154
</form>
140155
);
141156
};
142-

0 commit comments

Comments
 (0)