Skip to content

Commit

Permalink
Merge pull request #13416 from gallettilance/ocpbugs-24252
Browse files Browse the repository at this point in the history
OCPBUGS-24252: fix subscription page fields
  • Loading branch information
openshift-merge-bot[bot] committed Dec 6, 2023
2 parents 6d947df + 08693a5 commit 37d8ab9
Showing 1 changed file with 3 additions and 3 deletions.
Expand Up @@ -85,7 +85,7 @@ export const OperatorHubSubscribeForm: React.FC<OperatorHubSubscribeFormProps> =

const { pathname: url } = useLocation();

const [roleARNText, setRoleARNText] = React.useState(null);
const [roleARNText, setRoleARNText] = React.useState('');
const { catalogNamespace, channel, pkg, tokenizedAuth, version } = getURLSearchParams();
const [targetNamespace, setTargetNamespace] = React.useState(null);
const [installMode, setInstallMode] = React.useState(null);
Expand Down Expand Up @@ -477,7 +477,7 @@ export const OperatorHubSubscribeForm: React.FC<OperatorHubSubscribeFormProps> =
!namespaceSupports(selectedTargetNamespace)(selectedInstallMode) ||
(selectedTargetNamespace && cannotResolve) ||
!_.isEmpty(conflictingProvidedAPIs(selectedTargetNamespace)) ||
(tokenizedAuth === 'AWS' && _.isNull(roleARNText));
(tokenizedAuth === 'AWS' && _.isEmpty(roleARNText));

const formError = () => {
return (
Expand Down Expand Up @@ -773,7 +773,7 @@ export const OperatorHubSubscribeForm: React.FC<OperatorHubSubscribeFormProps> =
aria-label={'role ARN'}
type="text"
value={roleARNText}
onChange={(value) => {
onChange={(_event, value) => {
setRoleARNText(value);
}}
/>
Expand Down

0 comments on commit 37d8ab9

Please sign in to comment.