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

[Fix/#318] 방장 및 참여자 이름 입력 제한 글자수 8자로 변경 #319

Merged
merged 3 commits into from
Aug 31, 2024

Conversation

simeunseo
Copy link
Member

🌀 해당 이슈 번호

🔹 어떤 것을 변경했나요?

  • TextInput 컴포넌트에서 max값을 prop으로 받도록 변경
  • 방장 정보 입력 이름 제한 16자 -> 8자로 변경
  • 참여자 정보 입력 이름 제한 16자 -> 8자로 변경

🔹 어떻게 구현했나요?

기존에 TextInput 공통컴포넌트에서, 글자수 제한을 15자로 하드코딩되어있었습니다.
TextInput이 쓰이는 곳마다 글자수 제한을 다르게 줄 필요가 있기 때문에 이 값을 prop으로 추가했습니다.

그리고 참여자 정보 입력 input과 방장 정보 입력 input에서 max값을 8자로 넘겨주도록 구현했습니다.

🔹 PR 포인트를 알려주세요!

🔹 스크린샷을 남겨주세요!

2024-08-21.2.03.33.mov

@simeunseo simeunseo added fix fix 은서 은서의 개발 라벨 labels Aug 20, 2024
@simeunseo simeunseo requested a review from ljh0608 August 20, 2024 17:03
@simeunseo simeunseo self-assigned this Aug 20, 2024
@simeunseo simeunseo changed the title [Fix/#318] name validation [Fix/#318] 방장 및 참여자 이름 입력 제한 글자수 8자로 변경 Aug 20, 2024
Copy link
Member

@ljh0608 ljh0608 left a comment

Choose a reason for hiding this comment

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

max보다는 maxLength가 조금 더 좋은 네이밍이 아닐까 생각이 드는데 중요하진 않습니다!
굿입니다! 고생하셨어요!

@ljh0608 ljh0608 merged commit cd86765 into develop Aug 31, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix fix size/S 은서 은서의 개발 라벨
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Fix] 이름 제한 16자 -> 8자로 변경
2 participants