Skip to content
This repository was archived by the owner on Sep 30, 2024. It is now read-only.

Commit 4d69b06

Browse files
authored
remove extraneous Cody onboarding steps (#63373)
- Remove the "personal or work" step - Remove the "do you want us to call you?" step - Cleaner design of the Cody dashboard - Improve some wording - Remove CodySurveyToast (unused) and user.CompletedPostSignup. These have not been used in many months. On Amplitude, the relevant events have no data. Fix https://linear.app/sourcegraph/issue/PRIME-375/remove-personal-or-work-survey-signup-step <img width="2032" alt="image" src="https://github.com/sourcegraph/sourcegraph/assets/1976/ef100e60-71b5-479b-ac63-2450cdad0220"> ## Test plan n/a
1 parent 5413fd1 commit 4d69b06

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

53 files changed

+343
-2425
lines changed

client/shared/src/auth.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,6 @@ export const currentAuthStateQuery = gql`
4242
viewerCanAdminister
4343
tosAccepted
4444
hasVerifiedEmail
45-
completedPostSignup
4645
emails {
4746
email
4847
verified

client/shared/src/settings/temporary/TemporarySettings.ts

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -93,7 +93,6 @@ export interface TemporarySettingsSchema {
9393
'admin.hasCompletedLicenseCheck': boolean
9494
'simple.search.toggle': boolean
9595
'cody.onboarding.completed': boolean
96-
'cody.onboarding.step': number
9796

9897
/** OpenCodeGraph */
9998
'openCodeGraph.annotations.visible': boolean
@@ -161,7 +160,6 @@ const TEMPORARY_SETTINGS: Record<keyof TemporarySettings, null> = {
161160
'admin.hasCompletedLicenseCheck': null,
162161
'simple.search.toggle': null,
163162
'cody.onboarding.completed': null,
164-
'cody.onboarding.step': null,
165163
'openCodeGraph.annotations.visible': null,
166164
}
167165

client/shared/src/telemetry/event-names.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,6 @@ export const enum EventName {
1515
CODY_CHAT_SCOPE_INFERRED_REPO_DISABLED = 'web:codyChat:inferredRepoDisabled',
1616
CODY_CHAT_SCOPE_INFERRED_FILE_ENABLED = 'web:codyChat:inferredFileEnabled',
1717
CODY_CHAT_SCOPE_INFERRED_FILE_DISABLED = 'web:codyChat:inferredFileDisabled',
18-
VIEW_GET_CODY = 'GetCody',
1918

2019
CODY_EDITOR_WIDGET_VIEWED = 'web:codyEditorWidget:viewed',
2120
CODY_SIDEBAR_CHAT_OPENED = 'web:codySidebar:chatOpened',

client/shared/src/testing/integration/graphQlResults.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,6 @@ export const currentUserMock = {
2121
emails: [{ email: 'felix@sourcegraph.com', isPrimary: true, verified: true }],
2222
latestSettings: null,
2323
hasVerifiedEmail: true,
24-
completedPostSignup: true,
2524
permissions: {
2625
__typename: 'PermissionConnection',
2726
nodes: [

client/web-sveltekit/src/params/reporev.ts

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -33,12 +33,9 @@ const topLevelPaths = [
3333
'search/cody',
3434
'app',
3535
'cody',
36-
'get-cody',
37-
'post-sign-up',
3836
'unlock-account',
3937
'password-reset',
4038
'survey',
41-
'welcome',
4239
'embed',
4340
'users',
4441
'user',

client/web/BUILD.bazel

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -160,7 +160,6 @@ ts_project(
160160
"src/auth/AuthPageWrapper.tsx",
161161
"src/auth/CloudSignUpPage.tsx",
162162
"src/auth/OrDivider.tsx",
163-
"src/auth/PostSignUpPage.tsx",
164163
"src/auth/RequestAccessPage.tsx",
165164
"src/auth/ResetPasswordPage.tsx",
166165
"src/auth/SignInPage.tsx",
@@ -228,7 +227,6 @@ ts_project(
228227
"src/cody/components/ScopeSelector/index.tsx",
229228
"src/cody/components/ScopeSelector/useRepoSuggestions.ts",
230229
"src/cody/dashboard/CodyDashboardPage.tsx",
231-
"src/cody/editorGroups.ts",
232230
"src/cody/invites/AcceptInviteBanner.tsx",
233231
"src/cody/invites/InviteUsers.tsx",
234232
"src/cody/invites/useInviteParams.ts",
@@ -263,14 +261,6 @@ ts_project(
263261
"src/cody/management/subscription/manage/utils.ts",
264262
"src/cody/management/subscription/new/CodyProCheckoutForm.tsx",
265263
"src/cody/management/subscription/new/NewCodyProSubscriptionPage.tsx",
266-
"src/cody/onboarding/CodyOnboarding.tsx",
267-
"src/cody/onboarding/EditorStep.tsx",
268-
"src/cody/onboarding/PurposeStep.tsx",
269-
"src/cody/onboarding/WelcomeStep.tsx",
270-
"src/cody/onboarding/instructions/CodyFeatures.tsx",
271-
"src/cody/onboarding/instructions/JetBrains.tsx",
272-
"src/cody/onboarding/instructions/NeoVim.tsx",
273-
"src/cody/onboarding/instructions/VsCode.tsx",
274264
"src/cody/sidebar/CodySidebar.tsx",
275265
"src/cody/sidebar/Provider.tsx",
276266
"src/cody/sidebar/index.tsx",
@@ -1103,7 +1093,6 @@ ts_project(
11031093
"src/fuzzyFinder/SearchValue.ts",
11041094
"src/fuzzyFinder/SearchValueRankingCache.ts",
11051095
"src/fuzzyFinder/WordSensitiveFuzzySearch.ts",
1106-
"src/get-cody/GetCodyPage.tsx",
11071096
"src/global/GlobalAlert.tsx",
11081097
"src/global/GlobalAlerts.tsx",
11091098
"src/global/Notices.tsx",
@@ -1126,7 +1115,6 @@ ts_project(
11261115
"src/marketing/components/TweetFeedback.tsx",
11271116
"src/marketing/page/SurveyForm.tsx",
11281117
"src/marketing/page/SurveyPage.tsx",
1129-
"src/marketing/toast/CodySurveyToast.tsx",
11301118
"src/marketing/toast/SurveySuccessToast.tsx",
11311119
"src/marketing/toast/SurveyToastContent.tsx",
11321120
"src/marketing/toast/SurveyToastTrigger.tsx",

client/web/src/LegacyLayout.tsx

Lines changed: 4 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -98,15 +98,9 @@ export const LegacyLayout: FC<LegacyLayoutProps> = props => {
9898
const isSiteInit = location.pathname === PageRoutes.SiteAdminInit.toString()
9999
const isSignInOrUp =
100100
routeMatch &&
101-
[
102-
PageRoutes.SignIn,
103-
PageRoutes.SignUp,
104-
PageRoutes.PasswordReset,
105-
PageRoutes.Welcome,
106-
PageRoutes.RequestAccess,
107-
].includes(routeMatch as PageRoutes)
108-
const isGetCodyPage = location.pathname === PageRoutes.GetCody.toString()
109-
const isPostSignUpPage = location.pathname === PageRoutes.PostSignUp.toString()
101+
[PageRoutes.SignIn, PageRoutes.SignUp, PageRoutes.PasswordReset, PageRoutes.RequestAccess].includes(
102+
routeMatch as PageRoutes
103+
)
110104

111105
const [newSearchNavigation] = useNewSearchNavigation()
112106
const [enableContrastCompliantSyntaxHighlighting] = useFeatureFlag('contrast-compliant-syntax-highlighting')
@@ -228,7 +222,7 @@ export const LegacyLayout: FC<LegacyLayoutProps> = props => {
228222
telemetryRecorder={props.platformContext.telemetryRecorder}
229223
/>
230224
)}
231-
{!isSiteInit && !isSignInOrUp && !isGetCodyPage && !isPostSignUpPage && (
225+
{!isSiteInit && !isSignInOrUp && (
232226
<>
233227
{newSearchNavigation ? (
234228
<NewGlobalNavigationBar

client/web/src/auth/PostSignUpPage.tsx

Lines changed: 0 additions & 20 deletions
This file was deleted.

client/web/src/auth/SignUpPage.tsx

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -11,13 +11,12 @@ import { Container, Link, Text } from '@sourcegraph/wildcard'
1111
import type { AuthenticatedUser } from '../auth'
1212
import { PageTitle } from '../components/PageTitle'
1313
import type { SourcegraphContext } from '../jscontext'
14-
import { PageRoutes } from '../routes.constants'
1514
import { EventName } from '../util/constants'
1615

1716
import { AuthPageWrapper } from './AuthPageWrapper'
1817
import { CloudSignUpPage, ShowEmailFormQueryParameter } from './CloudSignUpPage'
1918
import { getReturnTo } from './SignInSignUpCommon'
20-
import { type SignUpArguments, SignUpForm } from './SignUpForm'
19+
import { SignUpForm, type SignUpArguments } from './SignUpForm'
2120
import { VsCodeSignUpPage } from './VsCodeSignUpPage'
2221

2322
import styles from './SignUpPage.module.scss'
@@ -92,8 +91,7 @@ export const SignUpPage: React.FunctionComponent<React.PropsWithChildren<SignUpP
9291
const v2Source = query.get('editor') === 'vscode' ? 0 : 1
9392
telemetryRecorder.recordEvent('auth.signUp', 'complete', { metadata: { source: v2Source } })
9493

95-
// Redirects to the /post-sign-up after successful signup on sourcegraphDotCom.
96-
window.location.replace(context.sourcegraphDotComMode ? PageRoutes.PostSignUp : returnTo)
94+
window.location.replace(returnTo)
9795

9896
return Promise.resolve()
9997
})

client/web/src/auth/__snapshots__/PostSignUpPage.test.tsx.snap

Lines changed: 0 additions & 41 deletions
This file was deleted.

0 commit comments

Comments
 (0)