Refactor: Reduce unnecessary map

[wit03]

🛠️ Refactor suggestion_

Refactor rendering logic to reduce duplication and improve type safety.

The current implementation handles both object and string types in the data.github array, which is flexible. However, this leads to code duplication and potential type safety issues. Consider refactoring to improve maintainability and type safety.

Here's a suggested refactor:

import { ReactNode } from 'react';

type GitHubItem = string | { url: string };

const GitHubLink = ({ href, children }: { href: string; children: ReactNode }) => (
  <Link
    href={href}
    target="_blank"
    className="flex items-center gap-2 rounded-lg bg-slate-50 px-6 py-4 min-w-72 flex-grow text-base font-medium line-clamp-2 hover:text-primaryRed"
  >
    <Image
      src="/logo/github.svg"
      alt="github logo"
      className="rounded-full mb-1 bg-white"
      width={30}
      height={24}
    />
    {children}
  </Link>
);

// In the component's return statement:
<div className="flex flex-row flex-wrap gap-4">
  {data.github.length === 0 ? (
    <p className="text-base font-normal text-gray-600">
      There is no github repository for this project.
    </p>
  ) : (
    data.github.map((item, i) => {
      const url = typeof item === 'string' ? item : item.url;
      return (
        <GitHubLink key={i} href={url}>
          {formatGithubLink(url)}
        </GitHubLink>
      );
    })
  )}
</div>

This refactoring:

1. Creates a reusable GitHubLink component.
2. Uses a type union GitHubItem to properly type the data.github array.
3. Simplifies the rendering logic by handling both cases in a single map function.
4. Improves readability and maintainability of the code.

Originally posted by @coderabbitai[bot] in #46 (comment)