fix(hybridcloud) Add org slug route for prompts-activity (#63122)

This endpoint is region scoped as all the current prompts are
organization bound. Having an organization_slug path will allow us to do
regional routing. I've also added the slug-less path to the US pinned
route list.

Once this is merged, I'll update the UI code to use the new route.

Refs HC-1066

Author: markstory

src/sentry/api/endpoints/prompts_activity.py

@@ -42,7 +42,7 @@ class PromptsActivityEndpoint(Endpoint):
     }
     permission_classes = (IsAuthenticated,)
 
-    def get(self, request: Request) -> Response:
+    def get(self, request: Request, **kwargs) -> Response:
         """Return feature prompt status if dismissed or in snoozed period"""
 
         features = request.GET.getlist("feature")
@@ -71,7 +71,7 @@ def get(self, request: Request) -> Response:
         else:
             return Response({"features": featuredata})
 
-    def put(self, request: Request):
+    def put(self, request: Request, **kwargs):
         serializer = PromptsActivitySerializer(data=request.data)
         if not serializer.is_valid():
             return Response(serializer.errors, status=400)

src/sentry/api/urls.py

@@ -2006,6 +2006,11 @@
         OrganizationUnsubscribeIssue.as_view(),
         name="sentry-api-0-organization-unsubscribe-issue",
     ),
+    re_path(
+        r"^(?P<organization_slug>[^/]+)/prompts-activity/$",
+        PromptsActivityEndpoint.as_view(),
+        name="sentry-api-0-organization-prompts-activity",
+    ),
 ]
 
 PROJECT_URLS: list[URLPattern | URLResolver] = [

src/sentry/apidocs/api_ownership_allowlist_dont_modify.py

@@ -275,6 +275,7 @@
     "/extensions/slack/action/",
     "/api/0/organizations/{organization_slug}/discover/saved/{query_id}/visit/",
     "/api/0/prompts-activity/",
+    "/api/0/organizations/{organization_slug}/prompts-activity/",
     "/api/0/teams/{organization_slug}/{team_slug}/avatar/",
     "/api/0/organizations/{organization_slug}/api-keys/{api_key_id}/",
     "/api/0/projects/{organization_slug}/{project_slug}/releases/completion/",

src/sentry/apidocs/api_publish_status_allowlist_dont_modify.py

@@ -681,6 +681,7 @@
     "/api/0/api-authorizations/": {"GET", "DELETE"},
     "/api/0/api-tokens/": {"DELETE", "GET", "POST"},
     "/api/0/prompts-activity/": {"GET", "PUT"},
+    "/api/0/organizations/{organization_slug}/prompts-activity/": {"GET", "PUT"},
     "/api/0/authenticators/": {"GET"},
     "/api/0/accept-transfer/": {"GET", "POST"},
     "/api/0/accept-invite/{organization_slug}/{member_id}/{token}/": {"GET", "POST"},

src/sentry/conf/server.py

@@ -3965,6 +3965,7 @@ def build_cdc_postgres_init_db_volume(settings: Any) -> dict[str, dict[str, str]
     # These paths have organization scoped aliases
     "sentry-api-0-builtin-symbol-sources",
     "sentry-api-0-grouping-configs",
+    "sentry-api-0-prompts-activity",
     # Legacy monitor endpoints
     "sentry-api-0-monitor-ingest-check-in-index",
     # These paths are used by relay which is implicitly region scoped

tests/sentry/api/endpoints/test_prompts_activity.py

@@ -8,17 +8,11 @@ def setUp(self):
         super().setUp()
         self.login_as(user=self.user)
         self.org = self.create_organization(owner=self.user, name="baz")
-        # self.project = self.create_project(
-        #     organization=self.org,
-        #     # teams=[self.team],
-        #     # name='Bengal-Elephant-Giraffe-Tree-House',
-        # )
-
         self.team = self.create_team(organization=self.org, name="Mariachi Band")
         self.project = self.create_project(
             organization=self.org, teams=[self.team], name="Bengal-Elephant-Giraffe-Tree-House"
         )
-        self.path = reverse("sentry-api-0-prompts-activity")
+        self.path = reverse("sentry-api-0-organization-prompts-activity", args=[self.org.slug])
 
     def test_invalid_feature(self):
         # Invalid feature prompt name
@@ -95,6 +89,10 @@ def test_dismiss(self):
         assert "data" in resp.data
         assert "dismissed_ts" in resp.data["data"]
 
+    def test_dismiss_legacy_path(self):
+        self.path = reverse("sentry-api-0-prompts-activity")
+        self.test_dismiss()
+
     def test_snooze(self):
         data = {
             "organization_id": self.org.id,
@@ -121,6 +119,10 @@ def test_snooze(self):
         assert "data" in resp.data
         assert "snoozed_ts" in resp.data["data"]
 
+    def test_snooze_legacy_path(self):
+        self.path = reverse("sentry-api-0-prompts-activity")
+        self.test_snooze()
+
     def test_batched(self):
         data = {
             "organization_id": self.org.id,