[Feat] 새로운 그룹 참여 방식 구현 #9
Conversation
js43o
force-pushed
the
group-join-api
branch
2 times, most recently
from
6a3e4aa
to
0a342a9
Compare
| <div className={styles.membersCount}> | ||
| <People width={16} height={16} fill={grayscale200} /> | ||
| <span>{membersCount}</span> | ||
| </div> |
There was a problem hiding this comment.
참여자 수는 그룹명 옆에 오게 할 수 있을까요?
app/frontend/src/services/group.ts
Outdated
| return data; | ||
| }, | ||
| join: async ({ id }: Pick<ResponseGroupsDto, 'id'>) => | ||
| morakAPI.post<null>(`${group.endPoint.default}/${id}/join`), | ||
| leave: async ({ id }: Pick<ResponseGroupsDto, 'id'>) => | ||
| morakAPI.delete<null>(`${group.endPoint.default}/${id}/leave`), | ||
| findByCode: async (accessCode: string) => { |
There was a problem hiding this comment.
p3; findByCode보다는 단일 그룹의 정보를 가져오는 거니까 group이라고만 해도 될 것 같다는 생각이 드네요.. (리액트 쿼리도) 서비스 부분의 컨벤션이 딱히 없어서 그런 것 같기도 하고요
There was a problem hiding this comment.
=> 5cda6d7 해당 API 경로(GET /groups/info)에 맞게 groupInfo로 맞추었습니다.
app/frontend/src/services/group.ts
Outdated
| `${group.endPoint.default}/my-groups`, | ||
| ); | ||
| const { data } = await morakAPI.get< | ||
| (ResponseGroupsDto & { membersCount: number; accessCode: string })[] |
There was a problem hiding this comment.
p2 : myGroup api의 타입은 ResponseMyGroupsDto를 사용하고 , all, findByCode api의 타입은 ResponseAccessCodeByGroupsDto 타입을 사용할 수 있을 것 같습니다.
There was a problem hiding this comment.
방금 콘솔 로그로 확인해보니 myGroup, findByCode의 DTO에 있는 memberCount가 실제 응답 데이터에서는 membersCount로 오고 있어서 해당 DTO를 쓸 수 없을 것 같습니다. 이번 주 회의에서 의견 나눈 후 반영해 보겠습니다!
There was a problem hiding this comment.
아 지금 공용패키지 dto에는 memberCount인데 서버 레포지토리단에서 membersCount로 반환하고 있네요.
memberCount로 수정하겠습니다.
There was a problem hiding this comment.
초대 코드 복사 기능이 필요한 그룹은 비공개 그룹인 거죠?
비공개 그룹은 그룹 목록에서 보이지 않는 거였나요?
그리고 초대 코드는 모든 회원이 볼 수 있는 건가요?
- 초대 코드는 공개, 비공개 그룹 모두 사용할 수 있게 했습니다. (진짜 '초대'를 위해 필요하다기 보다는 그룹 리스트에서 일일이 해당 그룹을 찾을 필요 없이 링크를 통해 바로 접근할 수 있게 해준다는 의미가 큰 것 같아요)
- 비공개 그룹도 공개 그룹처럼 리스트에 노출됩니다. (대신 바로 참여가 불가능하고 가입 신청만 가능)
- 초대 코드는 해당 그룹에 참여한 사용자에 한해서 마이페이지 - 내가 참여한 그룹 영역에서만 보이도록 했습니다.
의미가 조금 헷갈리는 것 같기도 해서 '초대 코드' 대신 '참여 링크'처럼 이름을 바꾸는 것도 좋겠다는 생각이 드네요..!
- 초대 코드 텍스트에 ellipsis 스타일 적용
- membersCount -> memberCount로 이름 변경
설명
완료한 기능 명세
스크린샷
bandicam.2024-02-25.22-20-07-529.mp4
리뷰 요청 사항