-
Notifications
You must be signed in to change notification settings - Fork 0
[Feat] 모임 목록 페이지 메타데이터 및 Open Graph 추가 #337
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the 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 |
🎨 Storybook Report✅ 변경 사항이 없습니다 모든 Story가 이전 빌드와 동일합니다.
|
🎭 Playwright Report✨ E2E Test가 성공적으로 완료되었습니다. Test 요약 내용을 확인해주세요.
📊 Test Summary
📜 Test Details✅ Passed Tests (3)
|
📊 Coverage Report
📉 #337을 main에 병합하면 coverage가 Coverage 요약@@ Coverage Diff @@
## main #337 +/- ##
===========================================
- Coverage 37.16% 36.85% -0.31%
===========================================
Files 228 229 +1
Lines 10589 10676 +87
Branches 420 421 +1
===========================================
Hits 3935 3935 0
+ Misses 6654 6741 +87 영향받은 파일✅ 이 PR로 영향받은 파일이 없습니다 수정된 모든 파일이 현재 coverage를 유지했습니다. |
🚀 PR Preview Report✨ Build가 성공적으로 완료되었습니다. Preview에서 변경사항을 확인하세요.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
오히려 좋아.
실무 프로젝트에서도 /root 경로의 컴포넌트 이름을 Home 또는 Main으로 가저가고 컴포넌트 경로에도 /components/home/... 으로 사용하곤 합니다!
(정답은 아니지만, 적어도 본인이 경험한 작고 소중한 두 곳은 그렇게 했었어용!)
P.S 메타몽 데이타는 해당 "페이지"의 정보를 web 로봇 벌레가 읽어가기 때문에 "기능"단의 네이밍이 아닌 "페이지"단의 이름을 사용하는 것이 옳다고 개인적으로 생각합니다 (뇌피셜)
|
위험한 버튼 누르고 싶다. |
📝 변경 사항
🔗 관련 이슈
Closes #
🧪 테스트 방법
📸 스크린샷 (선택)
📋 체크리스트
💬 추가 코멘트
CodeRabbit Review는 자동으로 실행되지 않습니다.
Review를 실행하려면 comment에 아래와 같이 작성해주세요