[Feat] 그룹 생성 페이지 추가 #509
Merged
[Feat] 그룹 생성 페이지 추가 #509
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- 임시 GroupCreate 타입 추가 - 공개/비공개 그룹 타입 useForm 사용해서 선택하도록 변경
js43o
approved these changes
Jan 28, 2024
| }; | ||
|
|
||
| return ['approve', 'code'].map((type) => ( | ||
| <label key={type} className={styles.inputField} htmlFor={type}> |
There was a problem hiding this comment.
p3: 저도 useController 훅을 사용하는 편이 html 쪽이 더 자연스럽고 보기 편해져서 좋은 것 같아요!
그리고 아까 그룹 방식 관련해서 잠깐 더 대화가 이어졌었는데, 참여 코드 방식이 아예 빠지고 비공개 그룹은 무조건 가입 신청->승인 방식으로만 진행하기로 다시 결론이 나왔던 것 같습니다.
LEEJW1953
approved these changes
Feb 3, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
설명
피그마
app/frontend/src/pages/Groups/Create/index.tsx파일에 작성한GroupJoin,GroupCreate타입은 API 명세 작성 전 임시로 작성한 타입입니다. 추후 변경 예정.체크박스, 라디오 버튼은 컴포넌트 추가 후 변경하겠습니다.
기존 react-hook-form의
Controller대신useController훅을 사용해 보았습니다.Controller를 사용하는 것보다 작성한 html 코드를 확인하는 것이 로직에 방해받지 않을 수 있어 더 깔끔하다고 생각하는데, 다른 분들 의견도 궁금합니다.완료한 기능 명세
스크린샷
2024-01-28.20.34.15.mov
리뷰 요청 사항