From 2ffa133683ce8e52de3162d9052ff2387862f132 Mon Sep 17 00:00:00 2001 From: cescoferraro Date: Fri, 16 Sep 2022 16:15:18 -0300 Subject: [PATCH] Revert "disable onboarding (#1211)" This reverts commit 858a836d6c67ebae436e030577553bab58bdeb00. --- web/cypress/support/commands.ts | 2 +- web/src/components/Header/HeaderMenu.tsx | 30 ++++++++++++++++++++---- 2 files changed, 26 insertions(+), 6 deletions(-) diff --git a/web/cypress/support/commands.ts b/web/cypress/support/commands.ts index e8f02d213b..ae3f7636e1 100644 --- a/web/cypress/support/commands.ts +++ b/web/cypress/support/commands.ts @@ -119,7 +119,7 @@ Cypress.Commands.add('cancelOnBoarding', () => { const parsedValue = value ? JSON.parse(value) : undefined; if (!parsedValue || parsedValue.trace === false) { - // cy.get('[data-cy=no-thanks]').click(); + cy.get('[data-cy=no-thanks]').click(); } }); diff --git a/web/src/components/Header/HeaderMenu.tsx b/web/src/components/Header/HeaderMenu.tsx index e9186dc4fb..6a553783a8 100644 --- a/web/src/components/Header/HeaderMenu.tsx +++ b/web/src/components/Header/HeaderMenu.tsx @@ -1,18 +1,38 @@ +import {useTour} from '@reactour/tour'; import {Popover, Typography} from 'antd'; +import {useMemo} from 'react'; +import {useLocation, useParams} from 'react-router-dom'; import {DOCUMENTATION_URL, GITHUB_URL} from 'constants/Common.constants'; -import {useLocation} from 'react-router-dom'; +import {useGuidedTour} from 'providers/GuidedTour/GuidedTour.provider'; +import HomeAnalyticsService from 'services/Analytics/HomeAnalytics.service'; import * as S from './Header.styled'; +import {ShowOnboardingContent} from './ShowOnboardingContent'; + +const {onGuidedTourClick} = HomeAnalyticsService; const HeaderMenu = () => { const {pathname} = useLocation(); + const params = useParams(); + const {setIsOpen} = useTour(); + const {isTriggerVisible, onCloseTrigger, setIsTriggerVisible} = useGuidedTour(); + + const content = useMemo( + () => + ShowOnboardingContent( + onGuidedTourClick, + () => setIsOpen(true), + () => onCloseTrigger() + ), + [onCloseTrigger, setIsOpen] + ); return ( null} + content={content} title={() => Take a quick tour of Tracetest?} - visible={false} + visible={isTriggerVisible} > { }, { key: 'Onboarding', - disabled: true, + disabled: !params.runId, label: ( - + setIsTriggerVisible(!isTriggerVisible)}> Show Onboarding ),