fix(web-domains, sds): 유저, 모임 생성, 모임 참여 1차 QA 수정 #116
Conversation
LeeJeongHooo
requested review from
Doeunnkimm,
semnil5202,
Andrevile and
sambad-adventure
as code owners
LeeJeongHooo
changed the title
| import { checkboxCss } from './styles'; | ||
|
|
||
| export interface CheckboxProps extends InputHTMLAttributes<HTMLInputElement> { | ||
| value: string | number; | ||
| label: string; | ||
| label: string | ((isChecked?: boolean) => ReactNode); |
There was a problem hiding this comment.
오 이거 왜 label이라는 프롭이 메서드까지 받는건지 알 수 있을까요? 👀
There was a problem hiding this comment.
기존에 Checkbox는 string만 받을 수 있었는데 qa에서 이모지랑 텍스트가 label에 모두 포함되어야 해서 그냥 컴포넌트를 내려받는 것도 포함시켜버렸습니다.
| export interface CheckboxLabelProps { | ||
| title: string; | ||
| isChecked?: boolean; | ||
| } | ||
|
|
There was a problem hiding this comment.
label에 htmlFor 달아놓으셨던데 그럼 label이 onClick 같은 이벤트 핸들러는 안 받는건가요 !?
음 현재 지면에서 사용할 일이 없으니 달지 않으셨던 거겠죠..?!
There was a problem hiding this comment.
위에서 설명한 부분 때문에 Checkbox에서 사용할 컴포넌트 용도로만 만들었던 거 같습니다. 사실 Label컴포넌트는 도은님 말씀대로 onClick이 있어야 되는 거 같습니다.
| required: '이름은 필수 입력 사항입니다', | ||
| minLength: { value: 2, message: '2자 이상 입력해주세요' }, | ||
| maxLength: { value: 10, message: '10자 이하로 입력해주세요' }, |
There was a problem hiding this comment.
흑흑 유효성 검사는 일괄적으로 한 파일에서 할 수 있도록 했으면 좋았을 거 같긴 합니다!
리팩터링 요소여서..패스해주셔도 상관 없긴 합니다.
There was a problem hiding this comment.
넵넵 나중에 리팩토링 할때 꼭 진행하겠습니다ㅠㅠ
| name="meetingTypeIds" | ||
| control={control} | ||
| rules={{ | ||
| validate: (value) => value.length <= 2 || '최대 2개까지 선택해주세요.', | ||
| validate: (value) => value.length <= 2 || '최대 2개까지 선택해주세요', | ||
| }} | ||
| render={({ field: { value, onChange } }) => ( | ||
| <CheckboxGroup value={value} onValueChange={onChange}> |
There was a problem hiding this comment.
호오.. register(Uncontrolled)랑 Controller(Controlled)를 혼용해서 사용하신 이유가 있나요?
| @@ -20,3 +20,9 @@ export const checkboxCss = { | |||
| borderColor: colors.primary500, | |||
| }, | |||
| }; | |||
|
|
|||
| export const itemCss = css({ | |||
| '& > * + *': { | |||
LeeJeongHooo
force-pushed
the
fix/user-get-info
branch
from
1ef36ab to
98a276b
Compare
🎉 변경 사항
🔗 링크
🙏 여기는 꼭 봐주세요!