//////
Search
🌅

1월 2일 월요일 (코드 리뷰)

생성자
회의 유형
임시 회의

마무리 해야할 일

게시글 수정 (은영)
좋아요 기능 (은영)
예외 처리 - 404, 데이터 없을 때, 로딩페이지 등(미경)
Post 여러장이미지 (미경)
데이터 없을 때 ⇒ 아직 작업 중 (미경)
이메일 회원가입 → 회원가입 하면 프로필 설정이 나오는데 새로고침 하면 이메일 회원가입으로 다시 로드되는 문제 (풍윤)
input custom require (풍윤)
실시간 폼 유효성 검증 (풍윤)
유저프로필에서 팔로우 버튼 기능(풍윤)
앨범 리스트 상세페이지 이동 추가
상품리스트 내꺼랑 친구꺼 동일하게 뜨는 문제

코드리뷰 전 확인

앨범형 이미지 세로 길게 보임
폰트 해결 (여러 이미지 캐러셀 엑박)
팔로잉 페이지에서 페이지를 찾을 수 없습니다 유저프로필 어카운트 주석? 없는 유저?
홈피드에서 게시글 신고 시 user_profile/undefined로 이동

호준님 코드리뷰 (3시~4시)

커밋 462 자신을 채찍질하며 하드하게 드라이브한 팀이다 ~~! 상위5퍼센트 탑 1이다 … 코드의 질은 커밋수를 따라가진 못했다 .. ^^
리코일
리코일로 authAtom 토큰을 저장해서 상태관리를 하려고 했는데, authAtom 값이 바로 안불러와지고 undefined으로 찍힌다. 예)로그인을 했을 때 바로 업데이트된 값을 못 받아오는 문제(my_profile/null) 새로고침하면 값이 불러와진다 why? 싱글페이지라서? 문제 전역으로 관리가 안되는 것 같다. 이렇게 사용하는 게 맞는지? 로컬스토리지로 관리하는 것과 차이를 모르겠다.
작은 단위의 컴포넌트
데이터를 받을 수 있는 상태로 깨끗하게 해두는 게 맞는지
동일한 이름으로 받는 게 맞는지
채팅방 전송은 안되더라도 입력하면 가는 것처럼 (gif로 하기 좋음)
캐러샐 버튼 투명도
스타일 컴포넌트 분리 추천
잘못된 통합보다는 중복이 낫다
.env 혹은 basicUrl 파일 하나 만들어서 공통 불러오기
통신 불러오는 것은 따로 빼기
회원가입 (-1)
로딩이라는 useState 만들어서 로딩이 되면 컴포넌트가 업데이트되게끔
로딩 성공하면 세 개 값을 할당받게끔
성능 최적화
signupUserProfile 140, 141줄 props 오타있어요
API에 모든 호출에 대한 걸 다 모아두기
지금 특별한 이유가 없고, localStorage.getItem(’auth’)만 필요한 곳에서 호출하면 된다. 아톰으로 저장할 필요없고 변수로 저장..
로컬스토리지 문제 : 개인정보가 들어가있으면 안 된다.(토큰값이나 쿠키값을 로컬스토리지에 저장하는 것은 예민한 문제다) 토큰값도 백엔 개발자랑 얘기해서 토큰값이 만료가 되는지, 시간이 지나면 아무 의미가 없는지 얘기할 필요가 있다. 로컬스토리지를 사용하지 않고 할 방법을 생각해보세요~
전역에서 state 값으로 관리할 수도 있고
15분 이력서 15분 코드 (다운받아서 … 볼 시간 없음) → 먼저 API 모아져있는지 확인, state는 시간 있으면 클릭해봄, 컴포넌트 필수(common -10점)
폴더 트리는 OK 호준님도 그렇게 짰을 것 같다
코드 개선은 나중에…. 단 수료하자마자 하세요 쉬지 마세요 ^^
배포보다 리드미 먼저
리드미 : 의사소통하는 데 문제는 없겠구나~ 문서 정리가 잘 되어있으면 가산점 (서류통과)
1번 프로젝트인데 정리가 안되어있으면 큰 마이너스
어떤 역할을 했는지 → 기술면접
개인 디벨롭 ㄴㄴ 팀플젝에서 계속 디벨롭 ㅇㅇ
80점 : API, 컴포넌트, 페이지는 ㄱㅊ
스타일드 컴포넌트가 안에 들어가있으면 감점! 스크롤 내리는데 드는 시간.. 스타일 코드가 짧아도 빼는 게 맞다.

해야할 일

리드미 (초안)
어플 소개 (무료 목업 이미지?)
데일리 스크럼을 했다 → 매일 9시 5시 상황 공유 하루 업무 공유
웹 스토리보드
역할 분담 작성
코드 트리 (은영)
역할 분담 (각자)
GIF (각자)
PPT (gif 위주로)
1차 배포
netlify
vercel

공통

API 분리
common 재사용 컴포넌트 분리
Base Url 이미지 파일 주소 `https://mandarin.api.weniv.co.kr/${res.data.filename}` -> 이건.. 필수 아님 ..
프로필 전역

추후 리팩토링 하고 싶은 부분

로그인 토큰 검증 (혜빈)
폼 밸리데이션 (풍윤)
채팅방 목업 데이터로 재구현 (은영)