8874343495

Committed 29 Apr 2024 06:36AM UTC coverage: 31.918% (-0.05%) from 31.971%

Build # 8874343495

Build Type

Pull #9190

github

Committed by

Stvad

Commit Message

Make EAOnboardingStage icon configurable

Pull Request Pull Request #9190: Generalize OnboardingFlow and introduce BasicOnboardingFlow for non-EA forums #to-upstream

Run Details

4709 of 38545 branches covered (12.22%)

Branch coverage included in aggregate %.

20 of 35 new or added lines in 6 files covered. (57.14%)

1 existing line in 1 file now uncovered.

27510 of 62397 relevant lines covered (44.09%)

179.71 hits per line

Source File
Press 'n' to go to next uncovered line, 'b' for previous

18.6

/packages/lesswrong/components/ea-forum/onboarding/useEAOnboarding.tsx

import React, { FC, ReactNode, createContext, useCallback, useContext, useState } from "react";
import { Components } from "../../../lib/vulcan-lib";
import { useCurrentUser, useRefetchCurrentUser } from "../../common/withUser";
import { UpdateCurrentUserFunction, useUpdateCurrentUser } from "../../hooks/useUpdateCurrentUser";
import { useTracking } from "../../../lib/analyticsEvents";

export type OnboardingStage = string

type EAOnboardingContext = {
  currentStage: OnboardingStage,
  goToNextStage: () => Promise<void>,
  goToNextStageAfter: <T>(promise: Promise<T>) => Promise<void>,
  nextStageIsLoading: boolean,
  currentUser: UsersCurrent,
  updateCurrentUser: UpdateCurrentUserFunction,
  captureOnboardingEvent: (type?: string, trackingData?: Record<string,any>) => void,
  // if viewAsAdmin is true, this is an admin testing out the flow, so don't update their account
  viewAsAdmin?: boolean,
}

const eaOnboardingContext = createContext<EAOnboardingContext>({
  currentStage: "",
  goToNextStage: async () => {},
  goToNextStageAfter: async () => {},
  nextStageIsLoading: false,
  currentUser: {} as UsersCurrent,
  updateCurrentUser: async () => {},
  captureOnboardingEvent: (type?: string, trackingData?: Record<string,any>) => {},
  viewAsAdmin: false,
});

export const EAOnboardingContextProvider: FC<{
  stages: Record<string, ReactNode>,
  onOnboardingComplete?: () => void,
  viewAsAdmin?: boolean,
}> = ({stages, onOnboardingComplete, viewAsAdmin}) => {
  const stageNames = Object.keys(stages)
  const [stage, setStage] = useState<OnboardingStage>(stageNames[0]);
  const [loading, setLoading] = useState(false);
  const currentUser = useCurrentUser();
  const updateCurrentUser = useUpdateCurrentUser();
  const refetchCurrentUser = useRefetchCurrentUser();
  const {captureEvent} = useTracking();

  const goToNextStage = useCallback(async () => {
    const getNextStage = (currentStage: OnboardingStage): OnboardingStage | undefined =>
      stageNames[stageNames.indexOf(currentStage) + 1]

    setLoading(true);
    await refetchCurrentUser();
    setLoading(false);
    const nextStage = getNextStage(stage);
    if (nextStage) {
      setStage(nextStage);
    } else {
      onOnboardingComplete?.();
    }
  }, [stage, onOnboardingComplete, refetchCurrentUser, stageNames]);

  const goToNextStageAfter = useCallback(async function<T>(promise: Promise<T>) {
    setLoading(true);
    await promise;
    await goToNextStage();
  }, [goToNextStage]);
  
  /**
   * Wrapper around captureEvent() that suppresses events when we are viewing as admin
   */
  const captureOnboardingEvent = useCallback((type?: string, trackingData?: Record<string,any>) => {
    !viewAsAdmin && captureEvent(type, trackingData);
  }, [viewAsAdmin, captureEvent])

  // This should never happen
  if (!currentUser) {
    const {LoginForm} = Components;
    return (
      <LoginForm />
    );
  }

  return (
    <eaOnboardingContext.Provider value={{
      currentStage: stage,
      goToNextStage,
      goToNextStageAfter,
      nextStageIsLoading: loading,
      currentUser,
      updateCurrentUser,
      captureOnboardingEvent,
      viewAsAdmin,
    }}>
      {...Object.values(stages)}
    </eaOnboardingContext.Provider>
  );
}

export const useEAOnboarding = () => useContext(eaOnboardingContext);

1	import React, { FC, ReactNode, createContext, useCallback, useContext, useState } from "react";	3✔
2	import { Components } from "../../../lib/vulcan-lib";	1✔
3	import { useCurrentUser, useRefetchCurrentUser } from "../../common/withUser";	1✔
4	import { UpdateCurrentUserFunction, useUpdateCurrentUser } from "../../hooks/useUpdateCurrentUser";	1✔
5	import { useTracking } from "../../../lib/analyticsEvents";	1✔
6
7	export type OnboardingStage = string
8
9	type EAOnboardingContext = {
10	currentStage: OnboardingStage,
11	goToNextStage: () => Promise<void>,
12	goToNextStageAfter: <T>(promise: Promise<T>) => Promise<void>,
13	nextStageIsLoading: boolean,
14	currentUser: UsersCurrent,
15	updateCurrentUser: UpdateCurrentUserFunction,
16	captureOnboardingEvent: (type?: string, trackingData?: Record<string,any>) => void,
17	// if viewAsAdmin is true, this is an admin testing out the flow, so don't update their account
18	viewAsAdmin?: boolean,
19	}
20
21	const eaOnboardingContext = createContext<EAOnboardingContext>({	1✔
22	currentStage: "",
23	goToNextStage: async () => {},
24	goToNextStageAfter: async () => {},
25	nextStageIsLoading: false,
26	currentUser: {} as UsersCurrent,
27	updateCurrentUser: async () => {},
28	captureOnboardingEvent: (type?: string, trackingData?: Record<string,any>) => {},
29	viewAsAdmin: false,
30	});
31
32	export const EAOnboardingContextProvider: FC<{
33	stages: Record<string, ReactNode>,
34	onOnboardingComplete?: () => void,
35	viewAsAdmin?: boolean,
36	}> = ({stages, onOnboardingComplete, viewAsAdmin}) => {	1✔
NEW 37	const stageNames = Object.keys(stages)	×
NEW 38	const [stage, setStage] = useState<OnboardingStage>(stageNames[0]);	×
39	const [loading, setLoading] = useState(false);	×
40	const currentUser = useCurrentUser();	×
41	const updateCurrentUser = useUpdateCurrentUser();	×
42	const refetchCurrentUser = useRefetchCurrentUser();	×
43	const {captureEvent} = useTracking();	×
44
45	const goToNextStage = useCallback(async () => {	×
NEW 46	const getNextStage = (currentStage: OnboardingStage): OnboardingStage \| undefined =>	×
NEW 47	stageNames[stageNames.indexOf(currentStage) + 1]	×
48
49	setLoading(true);	×
50	await refetchCurrentUser();	×
51	setLoading(false);	×
52	const nextStage = getNextStage(stage);	×
53	if (nextStage) {	×
54	setStage(nextStage);	×
55	} else {
56	onOnboardingComplete?.();	×
57	}
58	}, [stage, onOnboardingComplete, refetchCurrentUser, stageNames]);
59
60	const goToNextStageAfter = useCallback(async function<T>(promise: Promise<T>) {	×
61	setLoading(true);	×
62	await promise;	×
63	await goToNextStage();	×
64	}, [goToNextStage]);
65
66	/**
67	* Wrapper around captureEvent() that suppresses events when we are viewing as admin
68	*/
69	const captureOnboardingEvent = useCallback((type?: string, trackingData?: Record<string,any>) => {	×
70	!viewAsAdmin && captureEvent(type, trackingData);	×
71	}, [viewAsAdmin, captureEvent])
72
73	// This should never happen
74	if (!currentUser) {	×
75	const {LoginForm} = Components;	×
76	return (	×
77	<LoginForm />
78	);
79	}
80
81	return (	×
82	<eaOnboardingContext.Provider value={{
83	currentStage: stage,
84	goToNextStage,
85	goToNextStageAfter,
86	nextStageIsLoading: loading,
87	currentUser,
88	updateCurrentUser,
89	captureOnboardingEvent,
90	viewAsAdmin,
91	}}>
92	{...Object.values(stages)}
93	</eaOnboardingContext.Provider>
94	);
95	}
96
97	export const useEAOnboarding = () => useContext(eaOnboardingContext);	1✔

ForumMagnum / ForumMagnum / 8874343495

Source File Press 'n' to go to next uncovered line, 'b' for previous

Source File
Press 'n' to go to next uncovered line, 'b' for previous