Skip to content

Commit 01ed010

Browse files
committed
Make S3 region mandatory
1 parent 17c5d37 commit 01ed010

File tree

2 files changed

+4
-4
lines changed
  • src
    • entities/connection/api
    • features/connection/ConnectionDetailInfo/utils/getConnectionData

2 files changed

+4
-4
lines changed

src/entities/connection/api/types.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -250,9 +250,9 @@ export interface ConnectionS3 {
250250
host: string;
251251
bucket: string;
252252
bucket_style: ConnectionBucketStyle;
253-
port: number | null;
254-
region: string | null;
253+
region: string;
255254
protocol: ConnectionProtocol;
255+
port: number | null;
256256
};
257257
}
258258

src/features/connection/ConnectionDetailInfo/utils/getConnectionData/index.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -125,7 +125,7 @@ export const getConnectionData = (connection: Connection, t: TFunction<'connecti
125125
},
126126
{
127127
label: t('s3.region'),
128-
content: connection_data.region || '',
128+
content: connection_data.region,
129129
},
130130
];
131131
case ConnectionType.HIVE:
@@ -170,7 +170,7 @@ export const getConnectionData = (connection: Connection, t: TFunction<'connecti
170170
});
171171
result.push({
172172
label: t('s3.name') + ' ' + t('s3.region'),
173-
content: connection_data.s3_region || '',
173+
content: connection_data.s3_region,
174174
});
175175
} else {
176176
result.push({

0 commit comments

Comments
 (0)