From 45963b192993864543cad68f6791fddaebf78348 Mon Sep 17 00:00:00 2001 From: Alejandro Visiedo Date: Tue, 3 Sep 2024 13:15:00 +0200 Subject: [PATCH] fix(HMS-2141): update link and contents - Wizard Page 1 - Preparation: Remove paragraph about the pre-requisites and package verification, and leave only the 2nd and 3rd paragraph indicating the package to be installed. Remove the link about steps to install the server packages. Remove the link about pre-requisites. The link in the alert point out to documentation about "Deploying and managing RHEL systems in hybrid clouds". - Wizard page 2 - Registration: Remove the link in the warning alert. - The verify component remove the link to troubleshooting. - The cancel registration modal remove the link intended to provide information about deleting an ipa server, as the process is more related with the registration with the hcc. TODO - [ ] Feedback and comments to update the PR. https://issues.redhat.com/browse/HMS-2141 Signed-off-by: Alejandro Visiedo --- .../PagePreparation/PagePreparation.tsx | 62 ++++++++----------- .../PageServiceRegistration.tsx | 22 +------ .../VerifyRegistry/VerifyRegistry.tsx | 18 +----- src/Routes/WizardPage/WizardPage.tsx | 16 ----- 4 files changed, 28 insertions(+), 90 deletions(-) diff --git a/src/Routes/WizardPage/Components/PagePreparation/PagePreparation.tsx b/src/Routes/WizardPage/Components/PagePreparation/PagePreparation.tsx index dbd8406..9a2d3bb 100644 --- a/src/Routes/WizardPage/Components/PagePreparation/PagePreparation.tsx +++ b/src/Routes/WizardPage/Components/PagePreparation/PagePreparation.tsx @@ -24,8 +24,8 @@ interface PagePreparationProps { * @see {@link WizardPage} to know about the parent component. */ const PagePreparation = (props: PagePreparationProps) => { - // FIXME Update the target link when it is known - const installServerPackagesLink = 'https://duckduckgo.com/?q=freeipa+prerequisites'; + const deployAndManaingIPASystemsLink = + 'https://docs.redhat.com/en/documentation/red_hat_insights/1-latest/html-single/deploying_and_managing_rhel_systems_in_hybrid_clouds/index#managing-external-authentication-and-authorization-domains_host-management-services'; // States const appContext = useContext(AppContext); @@ -87,42 +87,32 @@ const PagePreparation = (props: PagePreparationProps) => { variant="info" isInline ouiaId="AlertPagePreparationPrepare" - > + > + + -
    -
  1. - - Verify whether or not the package is present on your Red Hat IdM server(s) by running the following command in a terminal on your Red - Hat IdM server(s): - - - dnf list installed ipa-hcc-server - - - If the package is not present on your Red Hat IdM server(s), follow these{' '} - - - dnf install ipa-hcc-server - - - - The package must be installed on at least one Red Hat IdM server. For redundancy, the package should be installed on two or more Red - Hat IdM servers. - -
    2. -
+ + Install the required components on your Red Hat IdM server(s): + + dnf install ipa-hcc-server + + + + The package must be installed on at least one Red Hat IdM server. For redundancy, the package should be installed on two or more Red Hat + IdM servers. + diff --git a/src/Routes/WizardPage/Components/PageServiceRegistration/PageServiceRegistration.tsx b/src/Routes/WizardPage/Components/PageServiceRegistration/PageServiceRegistration.tsx index 5d5c558..a17674e 100644 --- a/src/Routes/WizardPage/Components/PageServiceRegistration/PageServiceRegistration.tsx +++ b/src/Routes/WizardPage/Components/PageServiceRegistration/PageServiceRegistration.tsx @@ -1,5 +1,5 @@ import React, { useState } from 'react'; -import { Alert, Button, ClipboardCopy, Flex, FlexItem, Form, TextContent, Title } from '@patternfly/react-core'; +import { Alert, ClipboardCopy, Flex, FlexItem, Form, TextContent, Title } from '@patternfly/react-core'; import './PageServiceRegistration.scss'; import VerifyRegistry, { VerifyState } from '../VerifyRegistry/VerifyRegistry'; @@ -29,26 +29,11 @@ interface PageServiceRegistrationProp { * @public */ const PageServiceRegistration = (props: PageServiceRegistrationProp) => { - // FIXME Update the URL with the location for docs - // const installServerPackagesLink = 'https://freeipa.org/page/Quick_Start_Guide'; const [state, setState] = useState('initial'); - // FIXME Clean-up when sure it is not needed - // const openInNewWindow = (url: string) => { - // window.open(url, '_blank'); - // }; - - // FIXME Clean-up when sure it is not needed - // const onInstallServerPackagesClick = () => { - // openInNewWindow(installServerPackagesLink); - // }; - const ipa_hcc_register_cmd = 'ipa-hcc register ' + props.token; const alertTitle = 'Register your identity domain'; - // FIXME Update the URL with the location for docs - const linkLearnMoreAbout = 'https://www.google.es/search?q=freeipa+registering+a+domain+service'; - const onChangeVerifyRegistry = (newState: VerifyState, domain?: Domain) => { setState(newState); if (props.onVerify) { @@ -66,11 +51,6 @@ const PageServiceRegistration = (props: PageServiceRegistrationProp) => {
e.preventDefault()}> Completing this step registers your identity domain, and cannot be undone from the wizard.{' '} -
- -
  1. diff --git a/src/Routes/WizardPage/Components/VerifyRegistry/VerifyRegistry.tsx b/src/Routes/WizardPage/Components/VerifyRegistry/VerifyRegistry.tsx index 823aa27..ab55187 100644 --- a/src/Routes/WizardPage/Components/VerifyRegistry/VerifyRegistry.tsx +++ b/src/Routes/WizardPage/Components/VerifyRegistry/VerifyRegistry.tsx @@ -5,7 +5,6 @@ import { Button, Icon, Stack, StackItem, TextContent } from '@patternfly/react-c import { CheckCircleIcon } from '@patternfly/react-icons/dist/js/icons/check-circle-icon'; import { ExclamationCircleIcon } from '@patternfly/react-icons/dist/js/icons/exclamation-circle-icon'; import { PendingIcon } from '@patternfly/react-icons/dist/js/icons/pending-icon'; -import { ExternalLinkAltIcon } from '@patternfly/react-icons/dist/js/icons/external-link-alt-icon'; import './VerifyRegistry.scss'; import { Domain, ResourcesApiFactory } from '../../../../Api/idmsvc'; @@ -96,7 +95,6 @@ interface VerifyRegistryFooterProps { } const VerifyRegistryFooter = (props: VerifyRegistryFooterProps) => { - const linkTroubleshootRegistration = 'https://www.google.com/search?q=freeipa+troubleshooting'; return ( <> {props.state == 'initial' && ( @@ -114,21 +112,7 @@ const VerifyRegistryFooter = (props: VerifyRegistryFooterProps) => { )} - {props.state == 'not-found' && ( - <> - - - )} + {props.state == 'not-found' && <>} {props.state == 'completed' && ( <>