feat(person): add tmdb- and imdb link on person detail page

[bogo22]

Description

This PR adds links to tmdb and imdb on the person detail page.

NOTE: This PR is a 1:1 follow-up of unmerged PR #2096 because I accidentally delete the repo. Sorry for that.

To-Dos

• Disclosed any use of AI (see our policy)
• Successful build pnpm build
• [not needed] Translation keys pnpm i18n:extract
• [not needed] Database migration (if required)

Screenshot (if UI-related)

Summary by CodeRabbit

• New Features
  • Person profiles now display external reference links from entertainment databases, added to the person header for quick access.
  • Links are intelligently filtered by content type so movies, TV shows, and person profiles show the most relevant external references (with person-specific link behavior where applicable).

[github-actions]

This pull request has merge conflicts. Please resolve the conflicts so the PR can be successfully reviewed and merged.

[M0NsTeRRR]

Hello @bogo22,
can you rebase your PR so we can merge it ?

[coderabbitai]

📝 Walkthrough

Walkthrough

ExternalLinkBlock adds a new 'person' media type and conditional rendering to exclude or alter certain external links for person entities. PersonDetails now renders ExternalLinkBlock to show person-specific external links in the UI.

Changes

Cohort / File(s)	Summary
External Link Block Enhancement src/components/ExternalLinkBlock/index.tsx	Adds `ExternalLinkType = 'movie'
Person Details Integration src/components/PersonDetails/index.tsx	Imports and renders ExternalLinkBlock within the PersonDetails header/layout to surface external links for person entities; minor layout/className adjustment.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Poem

🐇 I nibble code beneath the moonlight,
Added "person" links that feel just right.
Conditional paths hop into place,
Names and IDs find their space.
Hooray — external links now smile bright!

🚥 Pre-merge checks | ✅ 2

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately describes the main change: adding TMDb and IMDb links to the person detail page, which is the primary objective of the PR.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

🧹 Nitpick comments (1)

src/components/ExternalLinkBlock/index.tsx (1)

76-95: Consolidate duplicated IMDb link rendering into one branch.

The two blocks are identical except for title vs name, so this can be simplified to reduce branching drift.

♻️ Proposed simplification

-      {imdbId && mediaType !== 'person' && (
-        <a
-          href={`https://www.imdb.com/title/${imdbId}`}
-          className="w-8 opacity-50 transition duration-300 hover:opacity-100"
-          target="_blank"
-          rel="noreferrer"
-        >
-          <ImdbLogo />
-        </a>
-      )}
-      {imdbId && mediaType === 'person' && (
-        <a
-          href={`https://www.imdb.com/name/${imdbId}`}
-          className="w-8 opacity-50 transition duration-300 hover:opacity-100"
-          target="_blank"
-          rel="noreferrer"
-        >
-          <ImdbLogo />
-        </a>
-      )}
+      {imdbId && (
+        <a
+          href={`https://www.imdb.com/${mediaType === 'person' ? 'name' : 'title'}/${imdbId}`}
+          className="w-8 opacity-50 transition duration-300 hover:opacity-100"
+          target="_blank"
+          rel="noreferrer"
+        >
+          <ImdbLogo />
+        </a>
+      )}

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@src/components/ExternalLinkBlock/index.tsx` around lines 76 - 95, The JSX
duplicates two identical <a> blocks for IMDb differing only by URL segment;
simplify in ExternalLinkBlock by rendering a single anchor when imdbId is truthy
and computing the path segment based on mediaType (`const imdbSegment =
mediaType === 'person' ? 'name' : 'title'`) then use
href={`https://www.imdb.com/${imdbSegment}/${imdbId}`} while keeping the same
className, target, rel and <ImdbLogo />; remove the two duplicated conditional
branches.

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Nitpick comments:
In `@src/components/ExternalLinkBlock/index.tsx`:
- Around line 76-95: The JSX duplicates two identical <a> blocks for IMDb
differing only by URL segment; simplify in ExternalLinkBlock by rendering a
single anchor when imdbId is truthy and computing the path segment based on
mediaType (`const imdbSegment = mediaType === 'person' ? 'name' : 'title'`) then
use href={`https://www.imdb.com/${imdbSegment}/${imdbId}`} while keeping the
same className, target, rel and <ImdbLogo />; remove the two duplicated
conditional branches.

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: c776b6fd-1a02-4ab4-b463-51b8d1c15d09

📥 Commits

Reviewing files that changed from the base of the PR and between e25c1a5 and c02c48e.

📒 Files selected for processing (2)

• src/components/ExternalLinkBlock/index.tsx
• src/components/PersonDetails/index.tsx

[coderabbitai]

🧹 Nitpick comments (1)

src/components/PersonDetails/index.tsx (1)

281-289: Flatten this wrapper to reduce layout noise.

The outer flex container only wraps a single child. You can simplify this block without changing behavior.

♻️ Suggested cleanup

-          <div className="flex w-full items-center justify-center lg:justify-between">
-            <div className="mb-3 mt-3">
-              <ExternalLinkBlock
-                mediaType="person"
-                tmdbId={data.id}
-                imdbId={data.imdbId}
-              />
-            </div>
-          </div>
+          <div className="mb-3 mt-3 flex w-full items-center justify-center lg:justify-between">
+            <ExternalLinkBlock
+              mediaType="person"
+              tmdbId={data.id}
+              imdbId={data.imdbId}
+            />
+          </div>

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@src/components/PersonDetails/index.tsx` around lines 281 - 289, The outer
flex wrapper around the ExternalLinkBlock is unnecessary since it only contains
one child; remove the surrounding <div className="flex w-full items-center
justify-center lg:justify-between"> and its inner margin wrapper, and render
ExternalLinkBlock directly (keeping props mediaType="person", tmdbId={data.id},
imdbId={data.imdbId}) so layout and behavior remain the same but with less
markup.

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Nitpick comments:
In `@src/components/PersonDetails/index.tsx`:
- Around line 281-289: The outer flex wrapper around the ExternalLinkBlock is
unnecessary since it only contains one child; remove the surrounding <div
className="flex w-full items-center justify-center lg:justify-between"> and its
inner margin wrapper, and render ExternalLinkBlock directly (keeping props
mediaType="person", tmdbId={data.id}, imdbId={data.imdbId}) so layout and
behavior remain the same but with less markup.

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: cbda6d2f-d117-4b82-8a89-a28e5d10c266

📥 Commits

Reviewing files that changed from the base of the PR and between c02c48e and 442d097.

📒 Files selected for processing (1)

• src/components/PersonDetails/index.tsx

[bogo22]

Politely requesting review to get this merged.