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

[#23] Feat: 공통컴포넌트구현 - TextInput #24

Merged
merged 2 commits into from
Apr 29, 2024

Conversation

hyeonjinan096
Copy link
Collaborator

@hyeonjinan096 hyeonjinan096 commented Apr 26, 2024

💬 Issue Number

closes #23

🤷‍♂️ Description

작업 내용에 대한 설명
TextInput 컴포넌트 구현했습니다.
디자인이 확정되지 않아서 아래 이미지 정도로 구현했습니다.
불필요하거나 추가해야하는 사항이 있다면 말씀해주세요:)

📷 Screenshots

작업 결과물
image

👻 Good Function

팀원에게 공유하고 싶은 함수나 코드 일부

팀원이 코드리뷰 시 주의할 점 또는 말하고 싶은 점 특이사항
textInput 컴포넌트에 fowardRef를 사용하면서 다음메세지의 eslint 에러가 발생했습니다.
Component definition is missing display name
리액트 컴포넌트 이름이 있어야한다는 eslint 규칙에서 발생한 경고인데 displayName 속성을 설정해서 문제를 해결할 수 있었습니다.
-> TextInput.displayName = 'TextInput'; 코드 추가 해결

📋 Check List

PR 전 체크해주세요.

  • Merge 하는 브랜치가 올바른가?
  • 코딩컨벤션을 준수하는가?
  • PR과 관련없는 변경사항이 없는가?

📒 Remarks

현재 pr에서 class-variance-authority 설치가 안돼서 빌드가 안되는 거 같습니다.
이전 class-variance-authority 설치했던 pr이 머지되면 pull한 후 머지해야할 거 같습니다.

@hyeonjinan096 hyeonjinan096 added the ✨Feature 새로운 기능 추가 label Apr 26, 2024
@hyeonjinan096 hyeonjinan096 added this to the 2주차 스프린트 milestone Apr 26, 2024
@hyeonjinan096 hyeonjinan096 self-assigned this Apr 26, 2024
@HoberMin
Copy link
Contributor

현진님 빌드오류 생겼는데 확인한번 부탁드립니다 !

@hyeonjinan096
Copy link
Collaborator Author

hyeonjinan096 commented Apr 26, 2024

현재 pr에서 class-variance-authority 설치가 안돼서 빌드가 안되는 거 같습니다.
이전에 설치한 class-variance-authority 또 설치하는 건 불필요할 거 같아서 이전 칩 pr이 머지되면 pull한 후 머지해야할 거 같은데
이렇게 하는게 좋을까요?

@HoberMin
Copy link
Contributor

HoberMin commented Apr 26, 2024

의존성이 어차피 중복설치가 되는게 아니라서 설치해도 괜찮습니다 !
버전만 맞춰주면 좋을 것 같아요

Copy link
Collaborator

@ienrum ienrum left a comment

Choose a reason for hiding this comment

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

수고하셨습니다 현진님!

Copy link
Collaborator

@hj9118 hj9118 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다~!

Copy link
Contributor

@HoberMin HoberMin left a comment

Choose a reason for hiding this comment

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

고생하셨습니다 !

src/app/_components/TextInput/index.tsx Show resolved Hide resolved
@HoberMin
Copy link
Contributor

현진님 PR 제목 확인한번만 하고 Merge해주세요 :)

@hyeonjinan096 hyeonjinan096 changed the title feat: TextInput 컴포넌트 구현 [#23] Feat: TextInput 컴포넌트 구현 Apr 27, 2024
@hyeonjinan096 hyeonjinan096 merged commit 26743b7 into main Apr 29, 2024
3 checks passed
@hyeonjinan096 hyeonjinan096 deleted the #23/feature/TextInput-component/hj branch April 29, 2024 11:09
@hyeonjinan096 hyeonjinan096 changed the title [#23] Feat: TextInput 컴포넌트 구현 [#23] Feat: 공통컴포넌트구현 - TextInput May 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨Feature 새로운 기능 추가
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feat] TextInput 컴포넌트 구현
4 participants