[BOM-1168] feat: 랜딩 배너 추가 및 SEO 개선#272
Hidden character warning
Conversation
jaeyoung-kwon
left a comment
There was a problem hiding this comment.
🤖 PR Review
✅ 확실하게 수정이 필요한 항목을 찾지 못했습니다.
랜딩 리다이렉트 제거 및 LandingIntroBanner 추가 변경이 의도한 SEO 전략과 일치합니다. 신규 컴포넌트의 반응형 처리와 캐러셀 통합이 올바르며, CarouselSlide의 min-width: 0 추가도 flex overflow 수정으로 적절합니다. canonical 설정(/landing → https://www.bombom.news)과 sitemap 정리도 SEO 정책에 부합합니다.
🚨 0 Critical ·
📋 검증 과정
- Claude structured review 결과 중 확신도가 있는 항목만 정리했습니다.
- Critical/Major는 inline comment로 게시하고, Minor는 참고 항목으로 summary에 포함합니다.
- 자동 생성된 OpenAPI 타입 선언 파일과 lock 파일은 리뷰 대상에서 제외합니다.
🤖 Claude PR Review
Generated by Claude Code
| title: '봄봄 | 읽고 남기고 쌓는 뉴스레터 리딩 플랫폼', | ||
| }, | ||
| ], | ||
| links: [{ rel: 'canonical', href: 'https://www.bombom.news' }], |
There was a problem hiding this comment.
혹시 이 코드의 의도를 더 자세히 알 수 있을까요?!
AI에게 물어보니 links는 루트 페이지로 지정 되어있는데, content는 'index'로 설정 되어 있어서 루트랑 랜딩 페이지랑 충돌돼서 의도와 다른 동작이 될 수 있다고 해서요!
그래서 /landing을 검색 노출할 페이지가 아닌 내부 소개용으로만 운영할거면 content를 'noindex'로 수정하라고 제안하긴 하는데.. 이부분에 대해선 문외한이라.. 일단 제나의 의도를 듣고싶어용
There was a problem hiding this comment.
아하 원래 의도는 랜딩의 검색 노출을 막으려는 것보다는 첫 방문자 리다이렉트 때문에 크롤링 봇까지 랜딩으로 이동하는 문제를 정리하려는 것이었습니당
리다이렉트 때문에 봇이 기존에 진입했던 루트 대신에 랜딩을 표준 콘텐츠로 인식하는걸 막으려고, canonical을 /로 설정해서 / 페이지가 표준임을 알려주려고 했어유
그런데 상추 말씀을 듣고 보니, 이제 리다이렉트 로직이 사라졌으니 랜딩의 canonical을 굳이 /로 설정할 필요가 있나 싶기도 하네요.. 아예 랜딩도 독립된 콘텐츠로 인식하도록 canonical을 /landing 으로 설정하고 인덱싱도 가능하도록 해서 루트 페이지와 별개로 관리하면 어떨까 싶은데 상추 의견이 궁금합니다!!
[이전]
links: [{ rel: 'canonical', href: 'https://www.bombom.news' }],
[이후]
links: [{ rel: 'canonical', href: 'https://www.bombom.news/landing' }],
There was a problem hiding this comment.
아하 그게 더 자연스러운 것 같아요!
근데 첫 방문자 랜딩 리다이렉트 로직 제거하면 랜딩은 언제 어떻게 접속할 수 있는걸까요?
There was a problem hiding this comment.
아하 둘러보기 페이지에 랜딩으로 이동할 수 있는 배너를 추가했습니다! 그래서 관심있는 사용자는 배너를 클릭해서 이동할 수 있습니다. 이외에도 저희끼리 홍보할때 서비스 소개 페이지가 필요하면 랜딩 링크를 첨부하는 방식으로 접속을 유도할 수 있을 것 같습니당
그리고 이제 landing도 대표 URL의 리다이렉트 페이지가 아니라, 독립된 콘텐츠로 관리하게 돼서(canonical), 저희 서비스를 검색하면 결과에 보일 수도 있지 않을까 싶습니당 ㅎㅎ
There was a problem hiding this comment.
sanghee01
left a comment
There was a problem hiding this comment.
굳 수고 많으셨습니다! 역시 SEO의 제나...
📌 What
LandingIntroBanner) 추가/landing항목 제거❓ Why
/landing으로 강제 리다이렉트하는 로직이 검색 엔진 크롤러의 앱 내부 진입을 막아 주요 페이지가 인덱싱되지 않는 문제가 있었음🔧 How
_bombom.tsx에서 첫 방문자 랜딩 리다이렉트 로직 제거SlideCardList)에LandingIntroBanner컴포넌트 추가landing.tsx에canonical링크(https://www.bombom.news) 설정/landing항목 제거 (canonical이 대체)👀 Review Point
LANDING_VISITED_KEYlocalStorage 키가 다른 곳에서 참조되는 부분 없는지 확인LandingIntroBanner모바일/태블릿/데스크톱 레이아웃 QAhttps://www.bombom.news가 실제 도메인 라우팅과 일치하는지 확인