Skip to content

Commit fdee145

Browse files
committed
Make S3 region mandatory
1 parent 3f2d888 commit fdee145

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

src/entities/connection/ui/ConnectionTypeForm/components/ConnectionIceberg/ConnectionIcebergS3Direct.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@ export const ConnectionIcebergS3Direct = () => {
3636
<Radio.Button value="path">{t('s3.bucketStylePath')}</Radio.Button>
3737
</Radio.Group>
3838
</Form.Item>
39-
<Form.Item label={t('s3.region')} name={['connection_data', 's3_region']}>
39+
<Form.Item label={t('s3.region')} name={['connection_data', 's3_region']} rules={[{ required: true }]}>
4040
<Input size="large" placeholder="us-east-1" />
4141
</Form.Item>
4242

src/entities/connection/ui/ConnectionTypeForm/components/ConnectionS3/index.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ export const ConnectionS3 = () => {
3535
<Radio.Button value="path">{t('s3.bucketStylePath')}</Radio.Button>
3636
</Radio.Group>
3737
</Form.Item>
38-
<Form.Item label={t('s3.region')} name={['connection_data', 'region']}>
38+
<Form.Item label={t('s3.region')} name={['connection_data', 'region']} rules={[{ required: true }]}>
3939
<Input size="large" placeholder="us-east-1" />
4040
</Form.Item>
4141

0 commit comments

Comments
 (0)