팀원소개서 팀원 목록 #213

100Gyeon · 2022-03-06T12:29:41Z

⛓ Related Issues

close 팀원소개서 팀원 목록 #212

📋 작업 내용

멤버 팝업에 최대 4명까지만 보이도록 수정
멤버 팝업에 전체보기 버튼 추가
팀원 목록 페이지 추가
자잘한 UI 수정사항 반영
- 이슈 카드 우측 하단에 팀, 작성자 정보 있는 부분 UI 수정
- 너가소개서 / 팀원소개서 메인 엠티뷰 UI 수정

📌 PR Point

원래 멤버 확인할 때 profileId가 필요 없었는데, 팀원 목록 페이지 추가되면서 profileId가 필요해졌습니다. 그래서 teamMemberList: TeamMemberNoneId[];에서 teamMemberList: TeamMember[];로 수정했습니다.
서버 response 수정사항(profileId 추가된 부분)은 어진이가 배포한 거 확인했습니다.
팀원 목록 페이지에서 < 버튼 들어간 헤더 임시로 추가했고, 나중에 주영 언니가 공통 컴포넌트 만들면 바꿔서 적용할게요!

👀 스크린샷 / GIF / 링크

SeojinSeojin · 2022-03-06T12:30:08Z

울 웹쁜이 고생많았어 ! 여기서 미리보기로 보면서 쉬어~ 다른 웹쁜이들한테도 자랑해줘~

SeojinSeojin

주말인데 벌써 뷰 하나를 뚝딱 🥘 수고많았어유~!!

SeojinSeojin · 2022-03-06T12:31:00Z

src/presentation/pages/Team/Main/MemberPopup/index.tsx

 import { TeamMemberNoneId } from '@api/types/team';
+import { MAX_TEAM_MEMBER } from '@utils/constant';


상수로 뺀거 넘 조아요~!

좋아요~!

SeojinSeojin · 2022-03-06T12:31:44Z

src/presentation/pages/Team/Main/MemberPopup/style.ts

  background: ${COLOR.WHITE};
-  border-radius: 8.07692px;
-  border: 0.8px solid ${COLOR.GRAY_3};
+  border-radius: 8.08px;


소수점이 줄어들었넹 ㅌㅋㅋㅋㅋㅋ 👍😍

ㅋㅋㅋㅋㅋ 드디어 !! 😻

SeojinSeojin · 2022-03-06T12:35:35Z

src/presentation/pages/Team/Main/index.tsx

+  const memberNameList =
+    slicedMemberList && slicedMemberList.map((member) => member.profileName).join(', ');


텍스트 한 줄로 해도 좋지만 나중에 호스트 친구는 굵은 글씨로 표시해야 해서 이전 방법처럼 span으로 하는 것도 좋은 선택일듯혀..!!
호스트가 생겨도 그냥 맨 앞에 따로 붙여두면 되니까 상관 없으려나?

이거 나두 생각해 보니까 이전 방법이 더 좋은 것 같아서 되돌려놓을게 !! 땡큐😉

SeojinSeojin · 2022-03-06T12:38:44Z

src/presentation/pages/Team/Member/index.tsx

+  const { data: teamInfoData } = useQuery(['teamDetailData', teamID], () =>
+    api.teamService.getTeamInfo(Number(teamID)),
+  );


리액트쿼리 천재만재!!! 🥰🥰

서진이가 첨에 잘 써놓은 덕분 ㅎㅎ😘

SeojinSeojin · 2022-03-06T12:39:45Z

src/presentation/routes/TeamRouter.tsx

 const TeamRegister = lazy(() => import('@pages/Team/Register'));
+const TeamMain = lazy(() => import('@pages/Team/Main'));
+const TeamMember = lazy(() => import('@pages/Team/Member'));


레이지로딩도 천재만재 🧠🧠

다른 애들 레이지로딩 되어있길래 알잘딱 맞춰놨으 ㅎㅎ

NamJwong

굳굳 진짜 짱 빨라 고생했어! 💓

NamJwong · 2022-03-06T12:35:54Z

src/presentation/components/common/Empty/HomeTeam/style.ts

-    ${FONT_STYLES.R_15_TITLE};
-    color: ${COLOR.GRAY_5};
+    ${FONT_STYLES.M_14_TITLE};
+    color: #dedede;


이거 컬러 상수 추가하는게 좋지 않을까?? 번호 35로 해서 하면 될 것 같은뎀 근데 기존에 만들어둔 번호가 30 40 300 400 이런 식이었으면 더 좋았을 . . (내가 만듦 ㅠ)

GRAY_35: '#DEDEDE'로 컬러 상수 추가했습니당 ㅎㅎ

굳굳 감사합니도 👍

NamJwong · 2022-03-06T12:56:19Z

src/presentation/pages/Team/Main/index.tsx

+  const slicedMemberList = teamInfoData && teamInfoData.teamMemberList.slice(0, MAX_TEAM_MEMBER);
+  const memberNameList =


이거 이름 slicedMemberNames로 하는거 어떨까? 일단 sliced를 붙이는게 더 이해하기 쉬울 것 같고 ~List는 배열 이름으로 쓰기로 해서 딱 봤을 때 배열 느낌이 너무 강한 것 같아서! 너낌 안오면 안바꿔도 됩니당

원래 코드로 되돌리기로 해서 이 변수는 안 쓰기로 했습니당 의견 고마워 ~~!

NamJwong · 2022-03-06T13:03:43Z

src/presentation/pages/Team/Member/index.tsx

+      <header>
+        <IcBack onClick={() => navigate(-1)} />
+        <div>팀원 목록</div>
+      </header>


이거 내가 공통 헤더 곧 만들어서 바꿔놓든지 할게!

좋아유~~! 🤩

SeojinSeojin · 2022-03-06T13:10:49Z

울 웹쁜이 고생많았어 ! 여기서 미리보기로 보면서 쉬어~ 다른 웹쁜이들한테도 자랑해줘~

SeojinSeojin · 2022-03-06T13:20:11Z

울 웹쁜이 고생많았어 ! 여기서 미리보기로 보면서 쉬어~ 다른 웹쁜이들한테도 자랑해줘~

100Gyeon added 13 commits March 6, 2022 17:11

🔨refactor: 매직 넘버 상수로 빼기

d6d3a0c

🎨design: UI 수정사항 반영

112f767

✨feat: 팀원 목록 페이지 Route 추가

e8410d0

✨feat: 팀원 목록 뷰 관련 세팅

9142ed9

🎨design: 이슈 카드 팀 정보 부분 UI 수정사항 반영

2da885b

✨feat: 팀원 목록 페이지 구조화 및 스타일링

a9d6d6d

🎨design: 팀 멤버 관련 부분 스타일링

dbc0849

🎨design: 너가소개서, 팀원소개서 메인 엠티뷰 UI 수정사항 반영

73f66c9

🎨design: 너가소개서 카드 UI 수정사항 반영 (border 추가)

8de827b

✨feat: 팀원 정보에 profileId 추가

e9c6181

📝chore: cursor 관련 설정 추가

148afb4

📝chore: 완료 버튼 없애기

325c5df

🔨refactor: 멤버 이름 4개만 보여주는 부분 수정

697d4d8

100Gyeon added feature 🎄 기능 개발 layout 🍃 레이아웃 개발 labels Mar 6, 2022

100Gyeon added this to the 3월 2주차 스프린트 milestone Mar 6, 2022

100Gyeon requested review from SeojinSeojin and NamJwong March 6, 2022 12:29

100Gyeon self-assigned this Mar 6, 2022

SeojinSeojin approved these changes Mar 6, 2022

View reviewed changes

NamJwong approved these changes Mar 6, 2022

View reviewed changes

📝chore: 이름 부분 원래대로 수정

2911b87

🔨refactor: 컬러 상수 추가

16e150a

100Gyeon merged commit e0539c4 into dev Mar 6, 2022

100Gyeon deleted the feat/#212 branch March 6, 2022 13:31

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

팀원소개서 팀원 목록 #213

팀원소개서 팀원 목록 #213

100Gyeon commented Mar 6, 2022

SeojinSeojin commented Mar 6, 2022

SeojinSeojin left a comment

SeojinSeojin Mar 6, 2022

NamJwong Mar 6, 2022

100Gyeon Mar 6, 2022

SeojinSeojin Mar 6, 2022

100Gyeon Mar 6, 2022

SeojinSeojin Mar 6, 2022

100Gyeon Mar 6, 2022

SeojinSeojin Mar 6, 2022

100Gyeon Mar 6, 2022

SeojinSeojin Mar 6, 2022

100Gyeon Mar 6, 2022

NamJwong left a comment

NamJwong Mar 6, 2022

100Gyeon Mar 6, 2022

NamJwong Mar 6, 2022

NamJwong Mar 6, 2022

100Gyeon Mar 6, 2022

NamJwong Mar 6, 2022

100Gyeon Mar 6, 2022

SeojinSeojin commented Mar 6, 2022

SeojinSeojin commented Mar 6, 2022

		import { TeamMemberNoneId } from '@api/types/team';
		import { MAX_TEAM_MEMBER } from '@utils/constant';

		const memberNameList =
		slicedMemberList && slicedMemberList.map((member) => member.profileName).join(', ');

		const slicedMemberList = teamInfoData && teamInfoData.teamMemberList.slice(0, MAX_TEAM_MEMBER);
		const memberNameList =

팀원소개서 팀원 목록 #213

팀원소개서 팀원 목록 #213

Conversation

100Gyeon commented Mar 6, 2022

⛓ Related Issues

📋 작업 내용

📌 PR Point

👀 스크린샷 / GIF / 링크

SeojinSeojin commented Mar 6, 2022

SeojinSeojin left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

NamJwong left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

SeojinSeojin commented Mar 6, 2022

SeojinSeojin commented Mar 6, 2022