From f4e332be56d72bec84ab73699f29e0e0da9cc2e6 Mon Sep 17 00:00:00 2001 From: fpasquet Date: Thu, 25 Apr 2024 14:50:45 +0200 Subject: [PATCH] fix: broken link on tutorial previous button (#1119) --- .../TutorialPageContainer/useTutorialPageContainer.tsx | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/src/containers/TutorialPageContainer/useTutorialPageContainer.tsx b/src/containers/TutorialPageContainer/useTutorialPageContainer.tsx index 168dbf035..5fa0cf96b 100644 --- a/src/containers/TutorialPageContainer/useTutorialPageContainer.tsx +++ b/src/containers/TutorialPageContainer/useTutorialPageContainer.tsx @@ -13,9 +13,10 @@ export const useTutorialPageContainer = (tutorial: TutorialPageData): PostPagePr const postPageProps = usePostPage(tutorial); const { slug, step: currentStep } = useParams<{ slug: string; step: string }>(); + const firstStep = tutorial.steps[0]; const [currentTutorialStepIndex, currentTutorialStep] = Object.entries(tutorial.steps).find( ([, step]) => step.slug === currentStep - ) ?? [0, tutorial.steps[0]]; + ) ?? [0, firstStep]; const previousStep = tutorial.steps[Number(currentTutorialStepIndex) - 1]; const nextStep = tutorial.steps[Number(currentTutorialStepIndex) + 1]; @@ -29,7 +30,7 @@ export const useTutorialPageContainer = (tutorial: TutorialPageData): PostPagePr label: step.title, href: generatePath(PATHS.POST, { lang: i18n.language, slug, step: index > 0 ? step.slug : undefined }), })), - sectionActive: currentTutorialStep?.slug ?? tutorial.steps[0].slug, + sectionActive: currentTutorialStep?.slug ?? firstStep.slug, }, children: , previousLink: previousStep @@ -38,7 +39,7 @@ export const useTutorialPageContainer = (tutorial: TutorialPageData): PostPagePr href: generatePath(PATHS.POST, { lang: i18n.language, slug, - step: previousStep.slug === 'introduction' ? '' : previousStep.slug, + step: previousStep.slug !== firstStep.slug ? previousStep.slug : undefined, }), } : undefined,