-
Notifications
You must be signed in to change notification settings - Fork 6
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
Feat: 하단 네브바 공통 컴포넌트 구현 #255
Feat: 하단 네브바 공통 컴포넌트 구현 #255
Conversation
* feat: layout 컴포넌트의 metadata 수정 - 리스티웨이브 한글 타이틀 추가 및 description 수정 * Feat: 마이리스트, 콜라보리스트 페이지에 Dynamic 메타데이터 설정 * Feat: 리스트 상세 페이지에 Dynamic 메타데이터 설정 * Refactor: 메타데이터 별도 상수로 분리 * Feat: sitemap 설정 및 도메인 url 별도 상수로 분리 * Feat: 트렌딩리스트에 해당하는 리스트의 상세페이지가 sitemap에 추가되도록 설정 * Feat: robots.txt 설정 * Fix: sitemap.ts 트랜딩리스트 호출 방법을 fetch API로 수정 * Fix: sitemap.ts 트랜딩리스트 호출 로직 제거
* feat: 라벨 및 카테고리 클릭시 검색페이지로 이동 * fix: 리스트 상세조회, 카테고리 전체조회 API 수정에 따라 타입 변경
- 숨기고 싶은 경로 대신 네브바가 보여지는 path로 변수 수정 - 네브바 탭 클릭시, 현재 위치를 표시하기 위한 분기처리 switch문으로 수정 - 네브바에서 콜렉션 탭 제거
@ParkSohyunee is attempting to deploy a commit to the Eujin Ahn's projects Team on Vercel. A member of the Team first needs to authorize it. |
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.
공통 컴포넌트 만들어주셔서 감사드립니다ㅠㅠ!!
디자인만 바꾼 것이 아닌 변경된 부분에 맞추어 코드 수정도 해주셔서 감사드려요!
가장 중요한 뼈대를 만들어주셨어요...🤍🙇♀️
const visiblePaths = ['/', '/mylist', 'collection']; | ||
const isVisible = visiblePaths.some((path) => pathname?.endsWith(path)); |
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.
하단 네브바가 depth 1에서만 보이니, 이렇게 보여지는 패스 설정이 훨씬 좋네요!!
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.
소현님 확인이 늦어서 죄송합니다 🥹 하단 nav바 만들어주셔서 정말 감사합니다! 🥰
개요
작업 사항
참고 사항 (optional)
스크린샷
리뷰어에게