Feat(landing client): meta description 변경#150
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
WalkthroughHTML 헤드 메타데이터(OG 설명)와 파비콘 링크가 apps/client와 apps/landing의 index.html에서 수정되었고, MyBookmark.tsx에 Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Suggested labels
Suggested reviewers
Poem
Pre-merge checks and finishing touches❌ Failed checks (2 warnings)
✅ Passed checks (3 passed)
✨ Finishing touches
🧪 Generate unit tests
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. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (7)
apps/client/src/pages/myBookmark/MyBookmark.tsx (1)
104-105: 렌더 중console.log제거 또는 개발환경에서만 제한렌더마다 실행되어 노이즈/미세 성능 악화 우려. 이번 PR 목적(메타 설명 변경)과도 무관합니다.
적용 Diff(삭제):
- console.log(category);선택안(개발환경에서만, 변경 시 1회 로깅):
- console.log(category); + // 개발환경 전용 디버깅 + // useEffect(() => { + // if (import.meta.env.DEV) console.debug('[MyBookmark] category', category); + // }, [category]);추가로 위 선택안을 쓰려면 상단 import에
useEffect를 포함하세요:-import { useState } from 'react'; +import { useEffect, useState } from 'react';apps/client/index.html (2)
12-27: 소셜 메타 확장(선택)Twitter 카드 메타 추가를 고려하면 미리보기 품질이 향상됩니다.
추가 예:
<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" />
2-2: lang 속성 재검토(선택)문서 주요 언어가 한국어라면
lang="ko-KR"적용을 고려하세요. 접근성/SEO에 유리합니다.apps/landing/index.html (2)
12-27: 소셜 메타 확장(선택)Twitter 카드 메타 추가를 고려하세요(클라이언트와 동일).
2-2: lang 속성 재검토(선택)한국어 우선 서비스라면
lang="ko-KR"권장.apps/landing/src/assets/2_bookmark.json (2)
2-2: 레이어/프로젝트 이름 인코딩 깨짐(모지박) 징후
"nm": "ë��..."등 한글이 깨져 있습니다. 에디터/저장 시 UTF-8 보장 필요. 시각적 영향은 없더라도 유지보수/검색성 저하.
- 파일 인코딩을 UTF-8로 고정
- 레이어/에셋 이름을 ASCII 또는 정상 한글로 재저장
266-296: Base64 인라인 PNG — 외부 파일 분리 권장(선택적 리팩토링)apps/landing/src/assets/2_bookmark.json — 파일 크기 35,235바이트, data:image/png;base64 인라인 4건 확인. 인라인 이미지로 번들 크기 증가 및 캐시 비효율 우려 있으므로, 성능/캐싱 개선을 원하면 PNG를 외부 파일로 분리하고 Lottie assets의 u/p로 참조하도록 변경하세요.
📜 Review details
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (2)
apps/client/public/link_Thumbnail.pngis excluded by!**/*.pngapps/landing/public/link_Thumbnail.pngis excluded by!**/*.png
📒 Files selected for processing (4)
apps/client/index.html(1 hunks)apps/client/src/pages/myBookmark/MyBookmark.tsx(1 hunks)apps/landing/index.html(1 hunks)apps/landing/src/assets/2_bookmark.json(1 hunks)
🔇 Additional comments (4)
apps/landing/src/assets/2_bookmark.json (1)
109-111: Lottie 버전 호환성 확인(선택)
"v": "5.7.0"입니다. 런타임(Lottie Web/Player) 버전과 호환되는지 실제 화면에서 재생 여부 확인 권장.apps/landing/index.html (1)
5-5: 파비콘 경로 확인 — landing에 파일 존재, 배포 루트 검증 필요
- 파일: apps/landing/public/chippi_extension_popup.svg (발견). 동일 파일이 apps/client/public/chippi_extension_popup.svg에도 존재.
- index.html의 href="/chippi_extension_popup.svg"는 앱이 루트('/')로 서빙될 때만 유효하므로, 배포 루트(호스트 경로/서브패스)를 확인하거나 상대경로(./chippi_extension_popup.svg), 또는 빌드 시 PUBLIC_URL/BASE_PATH 사용으로 수정하세요.
apps/client/index.html (2)
5-5: 파비콘 경로 확인 — 파일은 리포지토리에 있으나 CI/CD 포함 여부 확인 필요
파일: apps/client/public/chippi_extension_popup.svg, apps/landing/public/chippi_extension_popup.svg.
조치: 절대경로(/chippi_extension_popup.svg)는 배포 루트에 파일이 있어야 동작합니다. 빌드/배포(또는 CI)에서 public 폴더가 루트로 복사되는지 또는 해당 파일이 아티팩트에 포함되는지 확인하거나, CI에 루트 복사 작업을 추가하거나 링크를 상대경로/번들 방식으로 변경하세요.
7-10: meta description 문구가 PR 목표와 불일치 — 확인 필요rg 스크립트가 출력되지 않아 레포에서 meta description을 찾지 못했습니다. apps 쪽 index.html들에서 description이 PR 목표 문구("Pinback - 가장 재미있게 북마크를 활용하는 방법")로 변경되었는지 직접 확인하거나 아래 명령으로 재검색하세요:
rg -n -S "<meta[^>]*name=['\"]description['\"][^>]*>" --glob 'apps/**'
| <meta | ||
| name="description" | ||
| content="Pinback — 북마크, 리마인드, 리워드까지 한 번에." | ||
| /> |
There was a problem hiding this comment.
meta description 문구가 PR 목표와 불일치
landing도 동일하게 이전 문구입니다. 클라이언트와 맞춰 업데이트하세요.
적용 Diff:
- <meta
- name="description"
- content="Pinback — 북마크, 리마인드, 리워드까지 한 번에."
- />
+ <meta
+ name="description"
+ content="Pinback - 가장 재미있게 북마크를 활용하는 방법"
+ />📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| <meta | |
| name="description" | |
| content="Pinback — 북마크, 리마인드, 리워드까지 한 번에." | |
| /> | |
| <meta | |
| name="description" | |
| content="Pinback - 가장 재미있게 북마크를 활용하는 방법" | |
| /> |
🤖 Prompt for AI Agents
In apps/landing/index.html around lines 7 to 10, the meta description still
contains the old copy ("Pinback — 북마크, 리마인드, 리워드까지 한 번에."); update this meta
description to match the PR goal and the client copy used elsewhere (sync the
content attribute with the client app's current meta description). Replace the
content value with the exact string from the client/master copy, ensure proper
HTML escaping if needed, and save the file so landing and client descriptions
are identical.
into feat/#143/lotti-favicon
|
✅ Storybook chromatic 배포 확인: |
jjangminii
changed the title
1 participant
📌 Related Issues
📄 Tasks
⭐ PR Point (To Reviewer)
📷 Screenshot
Summary by CodeRabbit