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

[REFACTOR] 홈, 보드 생성 페이지 리팩토링 및 테스트 코드 추가 #130

Merged
merged 4 commits into from
Oct 8, 2024

Conversation

junseublim
Copy link
Collaborator

👀 관련 이슈

✨ 작업한 내용

  • 홈페이지, 보드 생성 페이지 리팩토링을 진행했습니다.
  • 리팩토링 작업 내용
    • 비즈니스 로직은 훅으로 분리: useBoardName, useBoardAvailability
    • 유틸성 로직은 재사용 가능하도록 utils에 함수로 분리: copyToClipboard
    • 입력 validation 관련 처리를 훅으로 분리: useInputValidation
    • 네이밍 수정: 명확한 네이밍을 사용하려고 합니다.
    • 테스트 코드 추가: hook, util 함수들에 대해 테스트 코드 작성
      • 저희 팀에서 사용하는 방식으로 우선 작성해봤습니다.

🌀 PR Point

  • 이해 안되는 부분이나 더 나은 방법이 떠오르신다면 알려주세요!

🍰 참고사항

  • 네이밍에 대한 컨벤션이 정해지면 좋을 것 같습니다..! 추후 정리해서 공유드리겠습니다 ㅎㅎ

@junseublim junseublim requested a review from hwanheejung October 5, 2024 03:43
Copy link
Member

@hwanheejung hwanheejung left a comment

Choose a reason for hiding this comment

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

LGTM!

@junseublim junseublim merged commit b966742 into develop Oct 8, 2024
@junseublim junseublim deleted the refactor/home-page branch October 8, 2024 17:24
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