Skip to content
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

fix(build): fix build error from button component #8

Merged
merged 6 commits into from
Feb 17, 2024

Conversation

Bokdol11859
Copy link
Member

작업 내역

구현 내용 및 작업 했던 내역

  • button 컴포넌트 path 오류로 인한 배포 오류 수정
  • 컴포넌트 폴더구조 통일
  • ticker 페이지 폴더를 (main) 내부로 이동

PR 특이 사항

PR을 볼 때 주의깊게 봐야하거나 말하고 싶은 점

  • �실제 서비스 기능과 관련된 페이지 / 컴포넌트들은 (main) 내부에 들어가야합니다! 리뷰하면서 놓쳤네요ㅠ
  • 컨플릭트 해결하는 과정에서 커밋 내역이 살짝 꼬였습니다,, 중복 커밋이 생겨버렸네요

Checklist

PR 등록 전 확인할 점

  • PR 제목은 포맷과 내용 둘 다 알맞게 작성되었는가 (e.g., feat(user): add login page)
  • assignee가 본인으로 되어있고, label은 PR 주제에 맞게 추가했는가
  • description에 PR에 대해 구체적으로 설명했는가

@Bokdol11859 Bokdol11859 self-assigned this Feb 17, 2024
@Bokdol11859 Bokdol11859 merged commit e894ebb into main Feb 17, 2024
1 check failed
@JinleeJeong
Copy link
Member

@Bokdol11859

�실제 서비스 기능과 관련된 페이지 / 컴포넌트들은 (main) 내부에 들어가야합니다! 리뷰하면서 놓쳤네요ㅠ
이거는 왜 필요한거예요?

@Bokdol11859
Copy link
Member Author

필수적인 부분은 아니지만, 랜딩페이지에서 사용하는 Layout과 Main에서 사용하는 Layout의 차이가 있는 경우가 많기에 폴더를 나누었고, Main Service 내부에서는 Layout이 공통적으로 적용되는 경우가 많아서 그렇습니다!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants