Fix #6291 UI: Remove required condition for dbt s3 from ui (#6361)

This commit is contained in:
Shailesh Parmar 2022-07-27 03:49:39 +05:30 committed by GitHub
parent a23a9c84c1
commit d3977bcb29
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
3 changed files with 3 additions and 10 deletions

View File

@ -62,10 +62,7 @@ export type DbtS3Creds = Pick<
| 'endPointURL'
>;
export type DbtS3CredsReq = Pick<
DbtS3Creds,
'awsAccessKeyId' | 'awsSecretAccessKey' | 'awsRegion'
>;
export type DbtS3CredsReq = Pick<DbtS3Creds, 'awsRegion'>;
export interface DbtSourceTypes {
sourceType: DBT_SOURCES;

View File

@ -78,8 +78,6 @@ export const reqDBTHttpFields: Record<keyof DbtConfigHttp, string> = {
};
export const reqDBTS3Fields: Record<keyof DbtS3CredsReq, string> = {
awsAccessKeyId: 'AWS Access Key ID',
awsSecretAccessKey: 'AWS Secret Access Key',
awsRegion: 'AWS Region',
};

View File

@ -92,7 +92,7 @@ export const DBTS3Config: FunctionComponent<Props> = ({
<label
className="tw-block tw-form-label tw-mb-1"
htmlFor="aws-access-key-id">
{requiredField('AWS Access Key ID')}
AWS Access Key ID
</label>
<p className="tw-text-grey-muted tw-mt-1 tw-mb-2 tw-text-xs">
AWS Access Key ID.
@ -106,13 +106,12 @@ export const DBTS3Config: FunctionComponent<Props> = ({
value={dbtSecurityConfig?.awsAccessKeyId}
onChange={(e) => updateS3Creds('awsAccessKeyId', e.target.value)}
/>
{errors?.awsAccessKeyId && errorMsg(errors.awsAccessKeyId)}
</Field>
<Field>
<label
className="tw-block tw-form-label tw-mb-1"
htmlFor="aws-secret-access-key-id">
{requiredField('AWS Secret Access Key')}
AWS Secret Access Key
</label>
<p className="tw-text-grey-muted tw-mt-1 tw-mb-2 tw-text-xs">
AWS Secret Access Key.
@ -126,7 +125,6 @@ export const DBTS3Config: FunctionComponent<Props> = ({
value={dbtSecurityConfig?.awsSecretAccessKey}
onChange={(e) => updateS3Creds('awsSecretAccessKey', e.target.value)}
/>
{errors?.awsSecretAccessKey && errorMsg(errors.awsSecretAccessKey)}
</Field>
<Field>
<label className="tw-block tw-form-label tw-mb-1" htmlFor="aws-region">