Clean up feedback system and fix API endpoints

Author: GiselleNessi

apps/dashboard/src/@/analytics/report.ts

@@ -547,62 +547,3 @@ export function reportChainInfraRpcOmissionAgreed(properties: {
 }) {
   posthog.capture("chain infra checkout rpc omission agreed", properties);
 }
-
-// ----------------------------
-// SUPPORT FEEDBACK
-// ----------------------------
-
-/**
- * ### Why do we need to report this event?
- * - To track successful feedback submissions for closed support tickets
- * - To understand feedback ratings distribution
- * - To measure customer satisfaction with support interactions
- *
- * ### Who is responsible for this event?
- * @gisellechacon
- */
-export function reportSupportFeedbackSubmitted(properties: {
-  ticketId: string;
-  rating: number;
-  hasFeedback: boolean;
-  feedbackLength?: number;
-}) {
-  posthog.capture("support feedback submitted", properties);
-}
-
-/**
- * ### Why do we need to report this event?
- * - To track failed feedback submissions for closed support tickets
- * - To identify and debug production issues with feedback system
- * - To understand error patterns and improve reliability
- *
- * ### Who is responsible for this event?
- * @gisellechacon
- */
-export function reportSupportFeedbackFailed(properties: {
-  ticketId: string;
-  rating: number;
-  hasFeedback: boolean;
-  feedbackLength?: number;
-  errorMessage: string;
-  errorType: "validation" | "network" | "server" | "unknown";
-}) {
-  posthog.capture("support feedback failed", properties);
-}
-
-/**
- * ### Why do we need to report this event?
- * - To track when feedback status checks fail
- * - To identify issues with feedback verification system
- * - To understand reliability of feedback status API
- *
- * ### Who is responsible for this event?
- * @gisellechacon
- */
-export function reportSupportFeedbackStatusCheckFailed(properties: {
-  ticketId: string;
-  errorMessage: string;
-  errorType: "network" | "server" | "unknown";
-}) {
-  posthog.capture("support feedback status check failed", properties);
-}

apps/dashboard/src/app/(app)/team/[team_slug]/(team)/~/support/_components/SupportCaseDetails.tsx

@@ -4,13 +4,8 @@ import { useMutation, useQuery, useQueryClient } from "@tanstack/react-query";
 import { format } from "date-fns";
 import { ChevronDownIcon, StarIcon, UserIcon } from "lucide-react";
 import Link from "next/link";
-import { Component, type ReactNode, useState } from "react";
+import { useState } from "react";
 import { toast } from "sonner";
-import {
-  reportSupportFeedbackFailed,
-  reportSupportFeedbackStatusCheckFailed,
-  reportSupportFeedbackSubmitted,
-} from "@/analytics/report";
 import type { Team } from "@/api/team/get-team";
 import { Badge } from "@/components/ui/badge";
 import {
@@ -51,27 +46,9 @@ function SupportCaseDetails({ ticket, team }: SupportCaseDetailsProps) {
   const [statusCheckFailed, setStatusCheckFailed] = useState(false);
 
   // Helper function to handle status check errors consistently
-  const handleStatusCheckError = (error: unknown) => {
-    const errorMessage = error instanceof Error ? error.message : String(error);
-    const errorType = /network|fetch/i.test(errorMessage)
-      ? "network"
-      : /API Server error/i.test(errorMessage)
-        ? "server"
-        : /Internal server error/i.test(errorMessage)
-          ? "server"
-          : "unknown";
-
-    // Report analytics for status check failure
-    reportSupportFeedbackStatusCheckFailed({
-      ticketId: ticket.id,
-      errorMessage,
-      errorType,
-    });
-
+  const handleStatusCheckError = (_error: unknown) => {
     // Set degraded state for warning banner
     setStatusCheckFailed(true);
-
-    // Don't throw; let the query return a safe fallback to avoid retries and duplicate analytics
     return;
   };
 
@@ -127,15 +104,6 @@ function SupportCaseDetails({ ticket, team }: SupportCaseDetailsProps) {
       return result;
     },
     onSuccess: () => {
-      // Report successful feedback submission
-      reportSupportFeedbackSubmitted({
-        ticketId: ticket.id,
-        rating,
-        hasFeedback: Boolean(feedback?.trim()),
-        feedbackLength: feedback?.length || 0,
-      });
-
-      // Clean success notification
       toast.success("Thank you for your feedback!");
 
       setRating(0);
@@ -147,32 +115,17 @@ function SupportCaseDetails({ ticket, team }: SupportCaseDetailsProps) {
     onError: (err) => {
       const msg = err instanceof Error ? err.message : String(err ?? "");
       let message = "Failed to submit feedback. Please try again.";
-      let errorType: "validation" | "network" | "server" | "unknown" =
-        "unknown";
 
       if (/network|fetch/i.test(msg)) {
         message = "Network error. Please check your connection and try again.";
-        errorType = "network";
       } else if (
         /validation|Rating must be|Please select a rating/i.test(msg)
       ) {
         message = msg; // show precise user-facing validation error
-        errorType = "validation";
       } else if (/API Server error/i.test(msg)) {
         message = "Server error. Please try again later.";
-        errorType = "server";
       }
 
-      // Report failed feedback submission
-      reportSupportFeedbackFailed({
-        ticketId: ticket.id,
-        rating,
-        hasFeedback: Boolean(feedback?.trim()),
-        feedbackLength: feedback?.length || 0,
-        errorMessage: msg,
-        errorType,
-      });
-
       toast.error(message);
     },
   });
@@ -530,57 +483,4 @@ function TicketMessage(props: { message: SupportMessage }) {
   );
 }
 
-// Error boundary for debugging production issues
-class FeedbackErrorBoundary extends Component<
-  { children: ReactNode; ticketId: string },
-  { hasError: boolean; error?: Error }
-> {
-  constructor(props: { children: ReactNode; ticketId: string }) {
-    super(props);
-    this.state = { hasError: false };
-  }
-
-  static getDerivedStateFromError(error: Error) {
-    return { hasError: true, error };
-  }
-
-  componentDidCatch(_error: Error, _errorInfo: { componentStack: string }) {
-    // Report the error to analytics using status check failure event
-    // which doesn't include misleading feedback metrics
-    reportSupportFeedbackStatusCheckFailed({
-      ticketId: this.props.ticketId,
-      errorMessage: `Error boundary: ${_error.message}`,
-      errorType: "unknown",
-    });
-  }
-
-  render() {
-    if (this.state.hasError) {
-      return (
-        <div className="border-t p-6">
-          <div className="text-destructive text-sm">
-            <p className="font-medium">
-              Something went wrong with the feedback system.
-            </p>
-            <p className="text-xs mt-1">
-              Error: {this.state.error?.message || "Unknown error"}
-            </p>
-          </div>
-        </div>
-      );
-    }
-
-    return this.props.children;
-  }
-}
-
-// Wrapper component with error boundary
-export function SupportCaseDetailsWithErrorBoundary(
-  props: SupportCaseDetailsProps,
-) {
-  return (
-    <FeedbackErrorBoundary ticketId={props.ticket.id}>
-      <SupportCaseDetails {...props} />
-    </FeedbackErrorBoundary>
-  );
-}
+export { SupportCaseDetails };

apps/dashboard/src/app/(app)/team/[team_slug]/(team)/~/support/cases/[id]/page.tsx

@@ -2,7 +2,7 @@ import { notFound, redirect } from "next/navigation";
 import { getAuthToken } from "@/api/auth-token";
 import { getTeamBySlug } from "@/api/team/get-team";
 import { tryCatch } from "@/utils/try-catch";
-import { SupportCaseDetailsWithErrorBoundary } from "../../_components/SupportCaseDetails";
+import { SupportCaseDetails } from "../../_components/SupportCaseDetails";
 import { getSupportTicket } from "../../apis/tickets";
 
 export default async function TicketPage(props: {
@@ -34,10 +34,5 @@ export default async function TicketPage(props: {
     redirect(`/team/${params.team_slug}/~/support`);
   }
 
-  return (
-    <SupportCaseDetailsWithErrorBoundary
-      team={team}
-      ticket={ticketResult.data}
-    />
-  );
+  return <SupportCaseDetails team={team} ticket={ticketResult.data} />;
 }