Skip to content

Commit

Permalink
Made changes to show both error messages for create donor cases and r…
Browse files Browse the repository at this point in the history
…eissue new donor case
  • Loading branch information
lambeb committed Oct 28, 2024
1 parent 1be9226 commit 64016e9
Show file tree
Hide file tree
Showing 5 changed files with 12 additions and 16 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@ function CreateDonorCasesConfirmation(): ReactElement {
};
}
isLoading(false);
navigate(`/questionnaire/${questionnaire.name}`, { state: { donorCasesResponseMessage: res.data, donorCasesStatusCode: res.status, questionnaire: questionnaire, role: role } });
navigate(`/questionnaire/${questionnaire.name}`, { state: { section: "createDonorCases", responseMessage: res.data, statusCode: res.status, questionnaire: questionnaire, role: role } });
}
if (loading) {
return <ONSLoadingPanel />;
Expand Down Expand Up @@ -61,7 +61,7 @@ function CreateDonorCasesConfirmation(): ReactElement {
/>
<ONSButton
label="Cancel"
onClick={() => navigate(`/questionnaire/${questionnaire.name}`, { state: { donorCasesResponseMessage: "", donorCasesStatusCode: 0, questionnaire: questionnaire, role: "" } })} primary={false} />
onClick={() => navigate(`/questionnaire/${questionnaire.name}`, { state: { section: "createDonorCases", responseMessage: "", statusCode: 0, questionnaire: questionnaire, role: "" } })} primary={false} />
</>
)
}
Expand All @@ -70,4 +70,4 @@ function CreateDonorCasesConfirmation(): ReactElement {
);
}

export default CreateDonorCasesConfirmation;
export default CreateDonorCasesConfirmation;
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,7 @@ import ReissueNewDonorCase from "./sections/reissueNewDonorCase";
import ReissueNewDonorCaseSummary from "../reissueNewDonorCasePage/reissueNewDonorCaseSummary";

interface State {
section?: string;
questionnaire: Questionnaire | null;
responseMessage?: string;
statusCode?: number;
Expand All @@ -33,7 +34,7 @@ function QuestionnaireDetailsPage(): ReactElement {
const [loaded, setLoaded] = useState<boolean>(false);
const initialState = location || { questionnaire: null };
const { questionnaireName } = useParams();
const { responseMessage, statusCode, role } = location || { responseMessage: "", statusCode: 0, role: "" };
const { section, responseMessage, statusCode, role } = location || { section: "", responseMessage: "", statusCode: 0, role: "" };

useEffect(() => {
if (initialState.questionnaire === null) {
Expand Down Expand Up @@ -117,11 +118,11 @@ function QuestionnaireDetailsPage(): ReactElement {
{questionnaire.name}
</h1>

{responseMessage && statusCode && role && <CreateDonorCasesSummary donorCasesResponseMessage={responseMessage} donorCasesStatusCode={statusCode} role={role} />}
{responseMessage && statusCode && role && <ReissueNewDonorCaseSummary responseMessage={responseMessage} statusCode={statusCode} role={role} />}
{section === "createDonorCases" && responseMessage && statusCode && role && <CreateDonorCasesSummary donorCasesResponseMessage={responseMessage} donorCasesStatusCode={statusCode} role={role} />}
{section === "reissueNewDonorCase" && responseMessage && statusCode && role && <ReissueNewDonorCaseSummary responseMessage={responseMessage} statusCode={statusCode} role={role} />}
<QuestionnaireDetails questionnaire={questionnaire} modes={modes} />
{questionnaire.name.includes("IPS") && <CreateDonorCases questionnaire={questionnaire} />}
<ReissueNewDonorCase questionnaire={questionnaire} />
{questionnaire.name.includes("IPS") && <ReissueNewDonorCase questionnaire={questionnaire} />}
<CatiModeDetails questionnaireName={questionnaire.name} modes={modes} />
<CawiModeDetails questionnaire={questionnaire} modes={modes} />
<TotalmobileDetails questionnaireName={questionnaire.name} />
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,7 @@ function CreateDonorCases({ questionnaire }: Props): ReactElement {
</td>
<td className="ons-summary__values" colSpan={2}>
<Link to="/createDonorCasesConfirmation"
state={{ questionnaire: questionnaire, role: role }}
state={{ section: "createDonorCases", questionnaire: questionnaire, role: role }}
className="ons-summary__button"
aria-label={`Create donor cases for questionnaire ${questionnaire.name}`}>
Create cases
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -15,12 +15,6 @@ function ReissueNewDonorCase({ questionnaire }: Props): ReactElement {
<div className="ons-summary__group">
<h2 className="ons-summary__group-title">Reissue New Donor Case for User</h2>
<table className="ons-summary__items">
<thead className="ons-u-vh">
<tr>
<th>Detail</th>
<th>Output</th>
</tr>
</thead>
<tbody className="ons-summary__item">
<tr className="ons-summary__row ons-summary__row--has-values">
<td className="ons-summary__item-title">
Expand All @@ -39,6 +33,7 @@ function ReissueNewDonorCase({ questionnaire }: Props): ReactElement {
<Link
to="/reissueNewDonorCaseConfirmation"
state={{
section: "reissueNewDonorCase",
questionnaire: questionnaire,
user: user
}}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@ function ReissueNewDonorCaseConfirmation(): ReactElement {
};
}
isLoading(false);
navigate(`/questionnaire/${questionnaire.name}`, { state: { reissueNewDonorCaseResponseMessage: res.data, reissueNewDonorCaseStatusCode: res.status, questionnaire: questionnaire, user: user } });
navigate(`/questionnaire/${questionnaire.name}`, { state: { section: "reissueNewDonorCase", responseMessage: res.data, statusCode: res.status, questionnaire: questionnaire, role: user } });
}
if (loading) {
return <ONSLoadingPanel />;
Expand Down Expand Up @@ -61,7 +61,7 @@ function ReissueNewDonorCaseConfirmation(): ReactElement {
/>
<ONSButton
label="Cancel"
onClick={() => navigate(`/questionnaire/${questionnaire.name}`, { state: { reissueNewDonorCaseResponseMessage: "", reissueNewDonorCaseStatusCode: 0, questionnaire: questionnaire, user: "" } })} primary={false} />
onClick={() => navigate(`/questionnaire/${questionnaire.name}`, { state: { section: "reissueNewDonorCase", responseMessage: "", statusCode: 0, questionnaire: questionnaire, role: "" } })} primary={false} />
</>
)
}
Expand Down

0 comments on commit 64016e9

Please sign in to comment.