Skip to content

Conversation

@jungmyunggi
Copy link
Collaborator

🔨 테스크

Fragment 삭제

  • 이전에 작업할 당시에는 또 다른 자식요소가 들어올 수 있을까 싶어서 fragment를 모두 써서 컴포넌트를 생성해놨었는데
  • 지금 보니까 그럴 이유가 없는거 같아 fragment를 모두 삭제했습니다.

같은 파일에서 중복 import 제거

  • 동일 파일에서 중복된 import가 있어서 하나로 합쳤습니다.

readOnly 삽입

  • 리액트에서는 기본적으로 props가 readonly로 가정을 하고 컴파일을 하기 때문에 성능에는 큰 차이가 없지만 그래도 불변성을 조금이나마 강조하고 싶어서 props에 readonly를 모두 붙였습니다. 혹시 컨벤션이 불편하다 싶으시면 바로 지우고 다시 커밋하도록 하겠습니다.

📋 작업 내용

  • Admin 컴포넌트 코드 리팩토링

@jungmyunggi jungmyunggi requested a review from junyeokk January 8, 2025 12:44
@jungmyunggi jungmyunggi self-assigned this Jan 8, 2025
@jungmyunggi jungmyunggi added the 🔨 Refactor 리팩토링 (구조 변경) label Jan 8, 2025
Copy link
Collaborator

@junyeokk junyeokk left a comment

Choose a reason for hiding this comment

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

수고하셨습니다 !

@jungmyunggi jungmyunggi merged commit 778ba5e into main Jan 9, 2025
1 check passed
@jungmyunggi jungmyunggi deleted the refactor/admin-components-refactor branch January 9, 2025 11:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🔨 Refactor 리팩토링 (구조 변경)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants