Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

remove extraneous Cody onboarding steps #63373

Merged
merged 3 commits into from
Jun 24, 2024
Merged

remove extraneous Cody onboarding steps #63373

merged 3 commits into from
Jun 24, 2024

Conversation

sqs
Copy link
Member

@sqs sqs commented Jun 20, 2024

  • 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 chenkc805 and a team 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 requested a review from rrhyne June 24, 2024 20:12
- No more modal during the Cody onboarding flow
- Cleaner design of Cody dashboard
Copy link
Contributor

@rrhyne rrhyne left a comment

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
17 checks passed
@sqs sqs deleted the sqs/rm-cody-survey branch June 24, 2024 20:37
sqs added a commit to sourcegraph/cody that referenced this pull request Jun 25, 2024
After sourcegraph/sourcegraph#63373, the `/post-sign-up` indirection/redirection is no longer necessary.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants