팀원소개서 메인, 상세 페이지 수정사항 반영 #85
Conversation
100Gyeon
requested review from
SeojinSeojin,
NamJwong and
Hyoin-Kim
and removed request for
SeojinSeojin
|
울 웹쁜이 고생많았어 ! 여기서 미리보기로 보면서 쉬어~ 다른 웹쁜이들한테도 자랑해줘~ |
|
울 웹쁜이 고생많았어 ! 여기서 미리보기로 보면서 쉬어~ 다른 웹쁜이들한테도 자랑해줘~ |
| @@ -13,4 +14,5 @@ export interface TeamService { | |||
| getMyIssue(): Promise<TeamIssueData>; | |||
| getTeamIssue(teamID: string): Promise<TeamIssueData>; | |||
| getTeamInfo(teamID: string): Promise<TeamInfoData>; | |||
| getInviteInfo(): Promise<TeamInviteData>; | |||
There was a problem hiding this comment.
초대만 보는 get api가 있는게 아니거 초대한 팀의 아이디를 보내야 그 팀의 정보를 조회하는 api가 있는 것 같은데 서버랑 더 맞춰보고 이야기를 나눠보자!!
그리고 초대받은 팀이 아예 없을 때, 여러 팀에서 초대받았을 때도 고려를 해야 할 것 같아!!
| @@ -75,18 +75,29 @@ export const StTeamInfo = styled.div` | |||
| display: flex; | |||
| align-items: center; | |||
| justify-content: center; | |||
| margin-bottom: 18px; | |||
| margin-bottom: 14.83px; | |||
src/infrastructure/api/types/team.ts
Outdated
| export type TeamInfoData = { | ||
| teamID: string; | ||
| teamImage?: string; | ||
| teamName: string; | ||
| teamDescription: string; | ||
| teamMembers: TeamMembers[]; | ||
| teamMembers: TeamMember[]; |
There was a problem hiding this comment.
이거 개인적으로 배열에 ~s가 아니라 ~List 하기로 해서 바꾸고 싶은데 이미 여러 곳에서 쓰고 있겠지.....?
There was a problem hiding this comment.
우리 코드컨벤션 보면 배열쓸때 ~List로 통일하기로 하긴했었다..!!!
There was a problem hiding this comment.
바꾸기 힘들면 나중에 리펙토링때 수정해도 좋을것같아요!!
|
울 웹쁜이 고생많았어 ! 여기서 미리보기로 보면서 쉬어~ 다른 웹쁜이들한테도 자랑해줘~ |
|
울 웹쁜이 고생많았어 ! 여기서 미리보기로 보면서 쉬어~ 다른 웹쁜이들한테도 자랑해줘~ |
|
울 웹쁜이 고생많았어 ! 여기서 미리보기로 보면서 쉬어~ 다른 웹쁜이들한테도 자랑해줘~ |
|
울 웹쁜이 고생많았어 ! 여기서 미리보기로 보면서 쉬어~ 다른 웹쁜이들한테도 자랑해줘~ |
|
울 웹쁜이 고생많았어 ! 여기서 미리보기로 보면서 쉬어~ 다른 웹쁜이들한테도 자랑해줘~ |
|
울 웹쁜이 고생많았어 ! 여기서 미리보기로 보면서 쉬어~ 다른 웹쁜이들한테도 자랑해줘~ |
|
울 웹쁜이 지금 보니까 PR도 엄청 잘 썼네.. 타입 관련해서 작업하느라 수고 많았고 머지 컨플릭트 난 건 내가 수정해서 푸쉬했어~ 이따가 같이 힘내보자! |
⛓ Related Issues
📋 작업 내용
📌 PR Point
TeamMembers[]타입과TeamMember[]타입이 같은데 2개나 있어서TeamMember[]로 통일했습니다.그래서 이슈 카드 리스트 관련 코드는 전부
TeamMember[]타입으로 변경했습니다.변경 후 관련된 모든 파일에서 오류 없는 거 확인했습니다.
예전 PR에서
key값으로index안 쓰고 싶다고 했는데, api 명세서 확인하니id값을 제공해 줘서index대신id를 사용하는 방식으로 수정했습니다.멤버 팝업에서 프로필 이미지가 없을 때 처리하는 코드가 없어서 이번에 추가했습니다.
팀 초대 수락/거절하는 부분은 아직 get 요청만 mock data 적용했습니다.
post는 api 명세서가 나오면 작업하는 게 좋을 것 같아요.
그래서 수락/거절 버튼 누르면 초대 알림 사라지게끔만 구현해놨습니다. 참고해 주세요!
👀 스크린샷 / GIF / 링크
메인 페이지 : https://naegasogaeseo-dev.kro.kr/home/team
상세 페이지 : https://naegasogaeseo-dev.kro.kr/team/0