Skip to content

Conversation

@hyeonjin6530
Copy link
Member

📌 관련 이슈번호

🎟️ PR 유형

어떤 변경 사항이 있나요?

  • 새 기능 추가
  • 버그 수정
  • CSS 등 사용자 UI 디자인 변경
  • 리팩토링

Check List

  • 🔀 PR 제목의 형식을 잘 작성했나요?
  • 🏗️ yarn build는 성공했나요?
  • 🧹 불필요한 코드는 제거했나요? (TODO, 주석, clg... etc.)
  • ✅ 컨벤션을 지켰나요?
  • 💭 이슈는 등록했나요?
  • 🏷️ 라벨은 등록했나요?
  • 🙇‍♂️ 리뷰어를 지정했나요?

✅ Key Changes

이번 PR에서 작업한 내용을 간략히 설명해주세요
사이드바 메뉴를 제작하고 헤더와 연결하였습니다.

  1. 작업 내용

📢 To Reviewers

  • 이전 작업이 머지가 안된 상태에서 개발을 진행하다보니 이전 브랜치에서 계속 작업하다가 호다닥 옮겨서 올려봅니다!! 혹시라도 오류가 있다면 말씀해주세요!!
  • 그리고 관리자인지의 여부에 따라 관리자 메뉴를 보여주는 로직을 생각해두었는데 아직 관리자의 유무를 어떤 방식으로 확인하는지 찾을 수 없어서 일단 해당 로직은 제외하고 만들었습니다!

📸 스크린샷

image

🔗 참고 자료

@hyeonjin6530 hyeonjin6530 added the ✨ FEAT 기능 구현 label Feb 23, 2025
@hyeonjin6530 hyeonjin6530 self-assigned this Feb 23, 2025
Copy link
Collaborator

@daun-up daun-up left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@hyeonjin6530 hyeonjin6530 merged commit 82cf455 into develop Feb 23, 2025
2 checks passed
@hyeonjin6530
Copy link
Member Author

LGTM!

고마워용 다운님🥰

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

✨ FEAT 기능 구현

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[ Feat ] 모바일 사이드바 구현

3 participants