From e7e05a3b6156191a5b26e01a4471334deeebaa0d Mon Sep 17 00:00:00 2001 From: Priscila Oliveira Date: Fri, 21 Feb 2025 12:02:35 +0100 Subject: [PATCH] ref(quick-start): Update onboarding task logic to create a first project entry --- src/sentry/receivers/onboarding.py | 30 ++++++++++++------------------ 1 file changed, 12 insertions(+), 18 deletions(-) diff --git a/src/sentry/receivers/onboarding.py b/src/sentry/receivers/onboarding.py index d704d3064fac5d..54fff3d489362e 100644 --- a/src/sentry/receivers/onboarding.py +++ b/src/sentry/receivers/onboarding.py @@ -97,25 +97,11 @@ def record_new_project(project, user=None, user_id=None, **kwargs): ), ) - success = OrganizationOnboardingTask.objects.record( - organization_id=project.organization_id, - task=OnboardingTask.FIRST_PROJECT, - user_id=user_id, - status=OnboardingTaskStatus.COMPLETE, - project_id=project.id, - ) - if not success: - # Check if the "first project" task already exists and log an error if needed - first_project_task_exists = OrganizationOnboardingTask.objects.filter( - organization_id=project.organization_id, task=OnboardingTask.FIRST_PROJECT - ).exists() - - if not first_project_task_exists: - sentry_sdk.capture_message( - f"An error occurred while trying to record the first project for organization ({project.organization_id})", - level="warning", - ) + first_project_task_exists = OrganizationOnboardingTask.objects.filter( + organization_id=project.organization_id, task=OnboardingTask.FIRST_PROJECT + ).exists() + if first_project_task_exists: OrganizationOnboardingTask.objects.record( organization_id=project.organization_id, task=OnboardingTask.SECOND_PLATFORM, @@ -130,6 +116,14 @@ def record_new_project(project, user=None, user_id=None, **kwargs): project_id=project.id, ) try_mark_onboarding_complete(project.organization_id) + else: + OrganizationOnboardingTask.objects.record( + organization_id=project.organization_id, + task=OnboardingTask.FIRST_PROJECT, + user_id=user_id, + status=OnboardingTaskStatus.COMPLETE, + project_id=project.id, + ) @first_event_received.connect(weak=False)