Fix(landing client): 중복된 코드 수정 by jjangminii · Pull Request #152 · Pinback-Team/pinback-client

jjangminii · 2025-09-22T17:18:58Z

📌 Related Issues

관련된 Issue를 태그해주세요. (e.g. - close #25)

close [Feat] meta 정보 수정 #151

📄 Tasks

중복된 메타정보 수정

Summary by CodeRabbit

신규 기능
버그 수정
- Client와 Landing 페이지에서 중복된 파비콘 링크를 제거하여 중복 로딩 문제를 방지했습니다.
잡무(Chores)
- Landing 페이지의 문서 언어를 영어(en)에서 한국어(ko)로 변경해 한국어 환경에 맞게 최적화했습니다.

vercel · 2025-09-22T17:19:03Z

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Preview	Comments	Updated (UTC)
pinback-client-client	Ready	Preview	Comment	Sep 22, 2025 5:19pm
pinback-client-landing	Ready	Preview	Comment	Sep 22, 2025 5:19pm

coderabbitai · 2025-09-22T17:19:05Z

Walkthrough

apps/client/index.html에서 중복 파비콘 링크를 제거했고, apps/landing/index.html에서는 문서 언어를 한국어로 변경하고 중복 파비콘 링크를 제거했습니다.

Changes

Cohort / File(s)	Change Summary
파비콘 링크 정리 `apps/client/index.html`, `apps/landing/index.html`	중복된 파비콘 `<link rel="icon" ...>` 태그 1개 제거하여 단일 링크만 유지.
문서 언어 설정 업데이트 `apps/landing/index.html`	HTML 루트 태그를 `<html lang="en">`에서 `<html lang="ko">`로 변경.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Feat(landing client): meta description 변경 #150: 동일 파일들의 파비콘 링크 태그 변경과 충돌/연관 가능성.
Feat(client,landing): lotti변경 및 전송시 메타데이터 추가 #144: 파비콘 href 및 메타데이터 수정으로 본 PR의 파비콘 정리와 직접적 연관.

Suggested labels

fix

Suggested reviewers

jllee000
constantly-dev

Poem

작은 귀 파닥, 헤드 태그 정리 완료!
쌍둥이 파비콘은 하나로 합쳐졌고,
lang도 ko라 토끼맘이 편안해요.
깡총—문서는 더 또렷, 길도 분명!
오늘도 캐럿처럼 깔끔한 빌드 🥕✨

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Linked Issues Check	⚠️ Warning	이 PR은 링크된 이슈 중 #151(메타 정보 수정)의 목표를 충족하여 중복 파비콘 제거와 landing의 lang 속성 변경을 포함하고 있습니다 (#151). 그러나 #25(디자인 시스템: Progress 컴포넌트 구현)의 핵심 요구사항(컴포넌트 구현, 스토리북·데모·런타임 의존성 추가)은 이 변경에 포함되어 있지 않습니다 (#25), 따라서 링크된 모든 이슈의 요구사항을 충족하지는 않습니다.	해결: PR이 실제로 #25를 다루지 않으면 linked_issues에서 #25를 제거하거나, #25를 이 PR에 포함하려면 Progress 컴포넌트 구현과 관련 스토리북·의존성 변경을 추가 커밋으로 포함하고 PR 설명을 업데이트해 주세요.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title Check	✅ Passed	제목 "Fix(landing client): 중복된 코드 수정"은 변경된 주요 내용(landing 및 client의 중복 메타/파비콘 제거)을 반영하여 변경사항과 관련성이 높습니다. 다만 '코드'라는 표현이 다소 일반적이라 어떤 중복인지(메타 정보/파비콘 등)를 명시하면 더 명확합니다.
Out of Scope Changes Check	✅ Passed	변경된 파일(apps/landing/index.html, apps/client/index.html)은 모두 메타 정보(파비콘 중복 제거, landing의 lang 변경) 관련 수정으로 보이며 PR 목적(중복된 메타정보 수정)과 직접 연관됩니다. 별도의 기능 추가나 다른 영역의 변경은 보이지 않아 범위 외 변경은 없습니다.
Description Check	✅ Passed	PR 설명에는 Related Issues( close #151 )와 Tasks(중복된 메타정보 수정)가 포함되어 있어 템플릿의 핵심 항목은 충족됩니다. 다만 템플릿의 'PR Point (To Reviewer)'와 'Screenshot' 섹션이 비어 있어 리뷰어에게 전달할 추가 컨텍스트나 변경 확인용 시각 자료가 필요하면 보완하는 것이 좋습니다.
Docstring Coverage	✅ Passed	No functions found in the changes. Docstring coverage check skipped.

✨ Finishing touches

🧪 Generate unit tests

Create PR with unit tests
Post copyable unit tests in a comment
Commit unit tests in branch feat/#151/meta-info-edit

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

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

github-actions · 2025-09-22T17:20:12Z

✅ Storybook chromatic 배포 확인:
🐿️ storybook

coderabbitai

Actionable comments posted: 0

🧹 Nitpick comments (4)

apps/landing/index.html (4)
5-5: 파비콘 중복 제거 OK — 호환성 확보용 아이콘 추가 제안

Safari/레거시 브라우저 호환과 PWA 품질을 위해 PNG/Apple Touch/Mask/Theme Color를 보강하는 것을 권장합니다.

아래 자산이 실제로 존재하면 추가하세요:
     <link rel="icon" type="image/svg+xml" href="/chippi_extension_popup.svg" />
+    <link rel="icon" type="image/png" sizes="32x32" href="/favicon-32x32.png" />
+    <link rel="icon" type="image/png" sizes="16x16" href="/favicon-16x16.png" />
+    <link rel="apple-touch-icon" sizes="180x180" href="/apple-touch-icon.png" />
+    <link rel="mask-icon" href="/safari-pinned-tab.svg" color="#000000" />
+    <meta name="theme-color" content="#ffffff" />
12-27: OG/Twitter 메타 보강 제안

한국어 지역 표기를 명시하고, 트위터 카드 미리보기도 함께 노출되도록 보강하면 좋습니다.
     <meta property="og:site_name" content="Pinback" />
     <meta property="og:title" content="Pinback" />
@@
     <meta property="og:image:height" content="400" />
     <meta property="og:image:alt" content="Pinback 미리보기 이미지" />
+    <meta property="og:locale" content="ko_KR" />
+    <meta name="twitter:card" content="summary_large_image" />
+    <meta name="twitter:title" content="Pinback" />
+    <meta name="twitter:description" content="Pinback - 가장 재미있게 북마크를 활용하는 방법" />
+    <meta name="twitter:image" content="https://pinback.today/link_Thumbnail.png" />
참고: 실제 og:image가 1200x630 등 권장 규격인지도 한 번 확인 부탁드립니다.

19-19: canonical 링크 추가 제안

중복 URL 정규화를 위해 canonical을 명시하면 SEO에 유익합니다.
     <meta property="og:url" content="https://pinback.today" />
+    <link rel="canonical" href="https://pinback.today" />
31-33: noscript 폴백 문구 추가 제안

JS 비활성 사용자/크롤러에 최소 안내를 제공합니다.
     <div id="root"></div>
+    <noscript>이 페이지를 보려면 JavaScript가 필요합니다.</noscript>
     <script type="module" src="/src/main.tsx"></script>

📜 Review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 029d6e8 and ffe7840.

📒 Files selected for processing (2)

apps/client/index.html (0 hunks)
apps/landing/index.html (1 hunks)

💤 Files with no reviewable changes (1)

apps/client/index.html

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)

GitHub Check: storybook

🔇 Additional comments (1)

apps/landing/index.html (1)

2-2: lang="ko" 변경 👍

접근성/SEO에 적절합니다. 다른 페이지들의 html lang도 ko(또는 대상 별 ko-KR)로 일관화됐는지 확인해주세요.

jjangminii added 2 commits September 23, 2025 02:16

Fix: 중복된 favicon 링크 제거

8f29fa7

feat: meta 정보 수정

ffe7840

jjangminii self-assigned this Sep 22, 2025

jjangminii linked an issue Sep 22, 2025 that may be closed by this pull request

[Feat] meta 정보 수정 #151

Closed

github-actions bot added the feat 기능 개발하라 개발 달려라 달려 label Sep 22, 2025

github-actions bot requested review from constantly-dev and jllee000 September 22, 2025 17:19

coderabbitai bot reviewed Sep 22, 2025

View reviewed changes

jjangminii merged commit 1ca0e74 into develop Sep 22, 2025
11 checks passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Fix(landing client): 중복된 코드 수정#152

Fix(landing client): 중복된 코드 수정#152
jjangminii merged 2 commits intodevelopfrom
feat/#151/meta-info-edit

jjangminii commented Sep 22, 2025 •

edited by coderabbitai bot

Loading

Uh oh!

vercel bot commented Sep 22, 2025

Uh oh!

coderabbitai bot commented Sep 22, 2025 •

edited

Loading

Uh oh!

github-actions bot commented Sep 22, 2025

Uh oh!

coderabbitai bot left a comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

Conversation

jjangminii commented Sep 22, 2025 • edited by coderabbitai bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

📌 Related Issues

📄 Tasks

Summary by CodeRabbit

Uh oh!

vercel bot commented Sep 22, 2025

Uh oh!

coderabbitai bot commented Sep 22, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Walkthrough

Changes

Estimated code review effort

Possibly related PRs

Suggested labels

Suggested reviewers

Poem

Pre-merge checks and finishing touches

Uh oh!

github-actions bot commented Sep 22, 2025

Uh oh!

coderabbitai bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

jjangminii commented Sep 22, 2025 •

edited by coderabbitai bot

Loading

coderabbitai bot commented Sep 22, 2025 •

edited

Loading