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

[FE][Fix][Refactor] : layout 폴더 src 디렉토리로 이동, AddChannel의 console.log 수정, 디렉토리 리팩토링 #428

Closed
wants to merge 6 commits into from

Conversation

effozen
Copy link
Collaborator

@effozen effozen commented Dec 4, 2024

📝 PR 개요

layout 폴더 밖으로 제거 + AddChannel의 console.log 수정

🔍 변경 사항

  • Layout 폴더 밖으로 제거
  • console.log 수정

✅ 체크리스트 (Checklist)

  • 코드가 빌드 오류 없이 잘 작동하는지 확인
  • 테스트가 통과하는지 확인
  • 스타일 가이드와 일관성을 유지했는지 확인
  • 관련 문서가 업데이트되었는지 확인 (선택 사항)
  • 리뷰어가 이해할 수 있도록 주석이나 설명을 추가했는지 확인

📷 스크린샷 및 동영상 (선택 사항)

2024-12-05.12.08.24.mov

🤔 고려 사항

  • Directory 를 다 리팩토링 하지는 못하지만, Layout 정도는 뺴면 좋을 것 같아서 빼두었습니다.
  • 이렇게 할 경우, 아래의 사진과 같은 프론트앤드 디렉토리 구조가 완성되어서, 1차적인 코드적 갖춤새는 갖추게 될 것 같아서 시도해봤습니다.
  • 다른 코드에 영향을 줄 수도 있는 부분이라 development 브랜치를 fork한 다른 브랜치를 이용했으며, 이에 따라서 문제가 될 것 같으면 이 PR을 close 하셔도 됩니다.
  • 어디까지나 제안사항입니다. 😁
  • 코드적으로 건든 부분은 없이 디렉토리만 이동하였고, 동작하는 것에 대한 테스트도 완료하였습니다.
스크린샷 2024-12-05 오전 12 32 41

@effozen effozen changed the title [FE][Fix][Refactor] : layout 폴더 src 디렉토리로 이동 + AddChannel의 console.log 수정 [FE][Fix][Refactor] : layout 폴더 src 디렉토리로 이동, AddChannel의 console.log 수정, 디렉토리 리팩토링 Dec 4, 2024
@github-actions github-actions bot added the 확인 요청 리뷰어에게 리뷰 요청 PR 날린 상태 (PR 머지 전) label Dec 4, 2024
@effozen effozen closed this Dec 5, 2024
@effozen effozen deleted the feature/fe/#000-refactor branch December 5, 2024 07:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
확인 요청 리뷰어에게 리뷰 요청 PR 날린 상태 (PR 머지 전)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants