apps/web/ui/partners/partner-info-cards.tsx (3)

38-46: Strongly type hideStatuses to partner status union.

Avoids typos and improves DX.

 export function PartnerInfoCards({
   partner,
-  hideStatuses = [],
+  hideStatuses = [],
   selectedGroupId,
   setSelectedGroupId,
 }: {
   partner?: EnrolledPartnerProps;
 
-  /** Partner statuses to hide badges for */
-  hideStatuses?: string[];
+  /** Partner statuses to hide badges for */
+  hideStatuses?: Array<EnrolledPartnerProps["status"]>;

---

122-126: Encode fallback avatar name to avoid broken URLs.

Names with spaces/special chars can yield invalid URLs.

-                src={partner.image || `${OG_AVATAR_URL}${partner.name}`}
+                src={partner.image || `${OG_AVATAR_URL}${encodeURIComponent(partner.name)}`}

---

143-147: Prefer omitting optional props over passing null.

If StatusBadge.icon is optional, drop icon={null} to avoid null checks downstream.

-            <StatusBadge icon={null} variant={badge.variant}>
+            <StatusBadge variant={badge.variant}>
               {badge.label}
             </StatusBadge>

Can you confirm icon is optional in @dub/ui’s StatusBadge?

apps/web/ui/partners/partner-application-details.tsx (2)

25-27: Prevent “undefined” in the heading before program loads.

Use a friendly fallback while program is loading.

-      title: `How do you plan to promote ${program?.name}?`,
+      title: `How do you plan to promote ${program?.name ?? "the program"}?`,

---

40-56: Don’t wrap placeholder text in Linkify.

Render the “No response provided” copy plainly; only linkify real responses.

-            {field.value || field.value === "" ? (
-              <Linkify
-                as="p"
-                options={{
-                  target: "_blank",
-                  rel: "noopener noreferrer nofollow",
-                  className:
-                    "underline underline-offset-4 text-neutral-400 hover:text-neutral-700",
-                }}
-              >
-                {field.value || (
-                  <span className="text-content-muted italic">
-                    No response provided
-                  </span>
-                )}
-              </Linkify>
-            ) : (
+            {field.value !== undefined ? (
+              field.value !== "" ? (
+                <Linkify
+                  as="p"
+                  options={{
+                    target: "_blank",
+                    rel: "noopener noreferrer nofollow",
+                    className:
+                      "underline underline-offset-4 text-neutral-400 hover:text-neutral-700",
+                  }}
+                >
+                  {field.value}
+                </Linkify>
+              ) : (
+                <p className="text-content-muted italic">No response provided</p>
+              )
+            ) : (
               <div className="h-4 w-28 min-w-0 animate-pulse rounded-md bg-neutral-200" />
             )}

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

Learnt from: TWilson023
PR: dubinc/dub#2736
File: apps/web/app/app.dub.co/(dashboard)/[slug]/(ee)/program/bounties/bounty-card.tsx:1-1
Timestamp: 2025-08-25T21:03:24.285Z
Learning: In Next.js App Router, Server Components that use hooks can work without "use client" directive if they are only imported by Client Components, as they get "promoted" to run on the client side within the Client Component boundary.