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] refactor: Input의 type에 올 수 있는 값을 html에서 제공하는 값으로 제한 #547

Merged
merged 1 commit into from
Sep 3, 2024

Conversation

ImxYJL
Copy link
Contributor

@ImxYJL ImxYJL commented Sep 1, 2024


🚀 어떤 기능을 구현했나요 ?

  • Input의 type 속성을 원래 단순 string으로만 명시했는데, 이렇게 하면 타입 이름을 잘못 입력했을 때 오류가 나지 않기 때문에 더 명확한 타입을 지정했습니다.

🔥 어떻게 해결했나요 ?

Input 인터페이스의 type을 이렇게 수정했습니다.
type: React.InputHTMLAttributes<HTMLInputElement>['type'];

📝 어떤 부분에 집중해서 리뷰해야 할까요?

  • 공통 Input을 사용하는 곳이 HomePage밖에 없어서 홈페이지의 input들이 잘 돌아가는지 확인했습니다. (정확히는 홈페이지 -> 리뷰 작성 완료까지 잘 돌아가는 걸 확인했습니다. 다른 곳에서는...안 쓰고 있겠죠?!)

📚 참고 자료, 할 말

Copy link
Contributor

@BadaHertz52 BadaHertz52 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

@chysis chysis 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

@soosoo22 soosoo22 left a comment

Choose a reason for hiding this comment

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

오호! 올리 덕분에 또 하나 배워갔네요!

@BadaHertz52 BadaHertz52 merged commit 33334b9 into develop Sep 3, 2024
5 checks passed
interface InputProps extends InputStyleProps {
type: React.InputHTMLAttributes<HTMLInputElement>['type'];
Copy link
Contributor

Choose a reason for hiding this comment

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

type: React.InputHTMLAttributes<HTMLInputElement>['type'];

type에 Input 속성 타입을 받아오도록 했고 props들도 늘어나서 이제 Input props들을 Input 속성 타입으로 변경해도 될 것 같네요.

@donghoony donghoony deleted the fe/refactor/546-specify-input-type branch September 26, 2024 02:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[FE] Input의 type 타입을 보다 명시적으로 정의한다.
4 participants