ref(nextjs-insights): Remove user option (BE)

src/sentry/users/api/endpoints/user_details.py

@@ -67,7 +67,6 @@ class UserOptionsSerializer(serializers.Serializer[UserOption]):
         required=False,
     )
     prefersIssueDetailsStreamlinedUI = serializers.BooleanField(required=False)
-    prefersNextjsInsightsOverview = serializers.BooleanField(required=False)
     prefersChonkUI = serializers.BooleanField(required=False)
 
 
@@ -237,7 +236,6 @@ def put(self, request: Request, user: User) -> Response:
             "defaultIssueEvent": "default_issue_event",
             "clock24Hours": "clock_24_hours",
             "prefersIssueDetailsStreamlinedUI": "prefers_issue_details_streamlined_ui",
-            "prefersNextjsInsightsOverview": "prefers_nextjs_insights_overview",
             "prefersChonkUI": "prefers_chonk_ui",
         }
 

src/sentry/users/api/serializers/user.py

@@ -73,7 +73,6 @@ class _UserOptions(TypedDict):
     timezone: str
     clock24Hours: bool
     prefersIssueDetailsStreamlinedUI: bool | None
-    prefersNextjsInsightsOverview: bool
     prefersChonkUI: bool | None
 
 
@@ -213,9 +212,6 @@ def serialize(
                 "defaultIssueEvent": options.get("default_issue_event") or "recommended",
                 "timezone": options.get("timezone") or settings.SENTRY_DEFAULT_TIME_ZONE,
                 "clock24Hours": options.get("clock_24_hours") or False,
-                "prefersNextjsInsightsOverview": options.get(
-                    "prefers_nextjs_insights_overview", True
-                ),
                 "prefersIssueDetailsStreamlinedUI": options.get(
                     "prefers_issue_details_streamlined_ui"
                 ),

src/sentry/users/models/user_option.py

@@ -164,8 +164,6 @@ class UserOption(Model):
         - unused
      - prefers_issue_details_streamlined_ui
         - Whether the user prefers the new issue details experience (boolean)
-    - prefers_nextjs_insights_overview
-        - Whether the user prefers the new NextJS insights overview experience (boolean)
      - prefers_chonk_ui
         - Whether the user prefers the new Chonk UI experience (boolean)
      - language

tests/sentry/users/api/endpoints/test_user_details.py

@@ -120,7 +120,6 @@ def test_simple(self) -> None:
                 "clock24Hours": True,
                 "extra": True,
                 "prefersIssueDetailsStreamlinedUI": True,
-                "prefersNextjsInsightsOverview": True,
                 "prefersChonkUI": True,
             },
         )
@@ -145,7 +144,6 @@ def test_simple(self) -> None:
             user=self.user, key="prefers_issue_details_streamlined_ui"
         )
         assert UserOption.objects.get_value(user=self.user, key="prefers_chonk_ui")
-        assert UserOption.objects.get_value(user=self.user, key="prefers_nextjs_insights_overview")
 
         assert not UserOption.objects.get_value(user=self.user, key="extra")
 
@@ -211,31 +209,6 @@ def test_cannot_change_username_to_non_verified(self) -> None:
         assert user.email == "c@example.com"
         assert user.username == "c@example.com"
 
-    def test_saving_nextjs_insights_overview_option(self) -> None:
-        self.get_success_response(
-            "me",
-            options={"prefersNextjsInsightsOverview": True},
-        )
-        assert (
-            UserOption.objects.get_value(user=self.user, key="prefers_nextjs_insights_overview")
-            is True
-        )
-
-        self.get_success_response(
-            "me",
-            options={"prefersNextjsInsightsOverview": False},
-        )
-        assert (
-            UserOption.objects.get_value(user=self.user, key="prefers_nextjs_insights_overview")
-            is False
-        )
-
-    def test_default_nextjs_insights_overview_option_is_true(self) -> None:
-        resp = self.get_success_response(
-            "me",
-        )
-        assert resp.data["options"]["prefersNextjsInsightsOverview"] is True
-
 
 @control_silo_test
 class UserDetailsSuperuserUpdateTest(UserDetailsTest):