feat(hackweek): Onboarding Tada Moment(s): Dopamine Driven Development

[priscilawebdev]

Goal

We made the onboarding too magical, and now it is kinda “meh” 🧙‍♂️

This project aims to gamify the onboarding experience to make it more fun and engaging. This will help motivate users and add more value to the process.

Previews

Project's Link
https://hackweek.getsentry.net/projects/-O10jVq3SmIDZ-a9WlrU/onboarding-tada-moments-dopamine-driven-development

[codecov]

❌ 3 Tests Failed:

Tests completed	Failed	Passed	Skipped
8020	3	8017	30

View the top 3 failed tests by shortest run time

EventOrGroupHeader Event lack of project adds all parameter EventOrGroupHeader Event lack of project adds all parameter

Stack Traces | 0.009s run time

Error: expect(element).toHaveAttribute("href", ".../groupID/events/eventID/?_allp=1&referrer=event-or-group-header") // element.getAttribute("href") === ".../groupID/events/eventID/?_allp=1&referrer=event-or-group-header"

Expected the element to have attribute:
  href=".../groupID/events/eventID/?_allp=1&referrer=event-or-group-header"
Received:
  href=".../groupID/events/eventID/?_allp=1&onboarding=3&referrer=event-or-group-header"
    at Object.<anonymous> (.../app/components/eventOrGroupHeader.spec.tsx:187:61)
    at Promise.then.completed (.../jest-circus/build/utils.js:298:28)
    at new Promise (<anonymous>)
    at callAsyncCircusFn (.../jest-circus/build/utils.js:231:10)
    at _callCircusTest (.../jest-circus/build/run.js:316:40)
    at processTicksAndRejections (node:internal/process/task_queues:95:5)
    at _runTest (.../jest-circus/build/run.js:252:3)
    at _runTestsForDescribeBlock (.../jest-circus/build/run.js:126:9)
    at _runTestsForDescribeBlock (.../jest-circus/build/run.js:121:9)
    at _runTestsForDescribeBlock (.../jest-circus/build/run.js:121:9)
    at run (.../jest-circus/build/run.js:71:3)
    at runAndTransformResultsToJestFormat (.../build/legacy-code-todo-rewrite/jestAdapterInit.js:122:21)
    at jestAdapter (.../build/legacy-code-todo-rewrite/jestAdapter.js:79:19)
    at runTestInternal (.../jest-runner/build/runTest.js:367:16)
    at runTest (.../jest-runner/build/runTest.js:444:34)
    at Object.worker (.../jest-runner/build/testWorker.js:106:12)

EventOrGroupHeader Event keeps sort in link when query has sort EventOrGroupHeader Event keeps sort in link when query has sort

Stack Traces | 0.024s run time

Error: expect(element).toHaveAttribute("href", ".../groupID/events/eventID/?_allp=1&referrer=event-or-group-header&sort=freq") // element.getAttribute("href") === ".../groupID/events/eventID/?_allp=1&referrer=event-or-group-header&sort=freq"

Expected the element to have attribute:
  href=".../groupID/events/eventID/?_allp=1&referrer=event-or-group-header&sort=freq"
Received:
  href=".../groupID/events/eventID/?_allp=1&onboarding=3&referrer=event-or-group-header&sort=freq"
    at Object.<anonymous> (.../app/components/eventOrGroupHeader.spec.tsx:169:61)
    at Promise.then.completed (.../jest-circus/build/utils.js:298:28)
    at new Promise (<anonymous>)
    at callAsyncCircusFn (.../jest-circus/build/utils.js:231:10)
    at _callCircusTest (.../jest-circus/build/run.js:316:40)
    at processTicksAndRejections (node:internal/process/task_queues:95:5)
    at _runTest (.../jest-circus/build/run.js:252:3)
    at _runTestsForDescribeBlock (.../jest-circus/build/run.js:126:9)
    at _runTestsForDescribeBlock (.../jest-circus/build/run.js:121:9)
    at _runTestsForDescribeBlock (.../jest-circus/build/run.js:121:9)
    at run (.../jest-circus/build/run.js:71:3)
    at runAndTransformResultsToJestFormat (.../build/legacy-code-todo-rewrite/jestAdapterInit.js:122:21)
    at jestAdapter (.../build/legacy-code-todo-rewrite/jestAdapter.js:79:19)
    at runTestInternal (.../jest-runner/build/runTest.js:367:16)
    at runTest (.../jest-runner/build/runTest.js:444:34)
    at Object.worker (.../jest-runner/build/testWorker.js:106:12)

ReleaseIssues includes release context when linking to issue ReleaseIssues includes release context when linking to issue

Stack Traces | 0.232s run time

Error: expect(element).toHaveAttribute("href", ".../org-slug/issues/123/?_allp=1&query=release%3A1.0.0&referrer=release-issue-stream") // element.getAttribute("href") === ".../org-slug/issues/123/?_allp=1&query=release%3A1.0.0&referrer=release-issue-stream"

Expected the element to have attribute:
  href=".../org-slug/issues/123/?_allp=1&query=release%3A1.0.0&referrer=release-issue-stream"
Received:
  href=".../org-slug/issues/123/?_allp=1&onboarding=3&query=release%3A1.0.0&referrer=release-issue-stream"
    at Object.<anonymous> (.../detail/overview/releaseIssues.spec.tsx:212:18)

To view individual test run time comparison to the main branch, go to the Test Analytics Dashboard

[github-actions]

🚨 Warning: This pull request contains Frontend and Backend changes!

It's discouraged to make changes to Sentry's Frontend and Backend in a single pull request. The Frontend and Backend are not atomically deployed. If the changes are interdependent of each other, they must be separated into two pull requests and be made forward or backwards compatible, such that the Backend or Frontend can be safely deployed independently.

Have questions? Please ask in the #discuss-dev-infra channel.

[priscilawebdev]

closing this since hackweek is over :)