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

remove extraneous Cody onboarding steps#63373

Merged
sqs merged 3 commits into
mainfrom
sqs/rm-cody-survey
Jun 24, 2024
Merged

remove extraneous Cody onboarding steps#63373
sqs merged 3 commits into
mainfrom
sqs/rm-cody-survey

Conversation

@sqs

@sqs sqs commented Jun 20, 2024

Copy link
Copy Markdown
Member
  • 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

image

Test plan

n/a

@cla-bot cla-bot Bot added the cla-signed label Jun 20, 2024
@sqs sqs force-pushed the sqs/rm-cody-survey branch 7 times, most recently from 625569e to e12aeba Compare June 24, 2024 18:21
@sqs sqs requested review from a team and chenkc805 June 24, 2024 18:22
@sqs sqs marked this pull request as ready for review June 24, 2024 18:23
These have not been used in many months.

On Amplitude, the relevant events have no data.
@sqs sqs force-pushed the sqs/rm-cody-survey branch from e12aeba to 6fa6142 Compare June 24, 2024 20:04
@sqs sqs requested a review from rrhyne June 24, 2024 20:12
- No more modal during the Cody onboarding flow
- Cleaner design of Cody dashboard
@sqs sqs force-pushed the sqs/rm-cody-survey branch from 6fa6142 to 9c4da91 Compare June 24, 2024 20:12

@rrhyne rrhyne left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lgtm, I’ll follow with some improvements and user tests.

@sqs sqs enabled auto-merge (squash) June 24, 2024 20:26
@sqs sqs merged commit 4d69b06 into main Jun 24, 2024
@sqs sqs deleted the sqs/rm-cody-survey branch June 24, 2024 20:37
sqs referenced this pull request in sourcegraph/cody-public-snapshot Jun 25, 2024
After https://github.com/sourcegraph/sourcegraph/pull/63373, the `/post-sign-up` indirection/redirection is no longer necessary.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants