search

modulersYJ / ganoverflow-front

2 stars 2 forks source link

`feat-chat-folderPost` : 사이드바 - 폴더/포스트 관련 update, delete, add 함수 등 적용, 외 2가지 #43

Closed ABizCho closed 1 year ago

ABizCho commented 1 year ago

@hongregii - 백엔드 먼저 main 머지하고 넣어야 deploy 안깨지긴 하는데, 그냥 이거 먼저 넣으셔도 돼요!

1. feat-routeModule: PATCH 추가 & POST, PUT, PATCH 추상화, 구조화된 props로 변경

routeModule의 수정사항들입니다.

1. PATCH 요청 함수를 추가헀습니다.

updateChatpostName(포스트이름 변경) 함수에서 PATCH요청을 사용하기 때문에 설계했습니다.

2. axios를 사용하는 세가지 요청함수 POST, PATCH, PUT 평가부 공통 추상화: REQUEST

3. axios 요청 함수 세가지 IProps 설계 및 구조화된 props 받도록 설정

  1. 명시적 key 준수에 따른, 순서자유도 및 선택적 인자를 본격적으로 사용할 수 있음.
  2. 이에 따른 부수작업: axios 요청 함수 호출 측 인자 입력 일괄 수정



2. feat-chat-folderPost : 사이드바 - 폴더/포스트 관련 update, delete, add 함수 등 적용

0. 사이드바 onClickNewFloderBtn 이벤트핸들러 구현 및 적용

  1. Default folder(ID=0), NewFolder, ...OtherFolders 순으로 저장
  2. setFoldersData로 state 업데이트 -> page.tsx의 put 트리거

1. 사이드바 Update PostName & Update FolderName 구현

  1. TitleEdit.tsx로 Name변경 컴포넌트 모듈화 하여 공통 적용
  2. updateChatpostName, updateFoldersWithPosts 함수 구현
  3. 위의 함수에서 활용될 PATCH 요청함수 추가 및 활용(직전 커밋)

2. 사이드바 Delete Post 구현

  1. 폴더에 딸린 삭제 버튼 클릭 시, 즉시 onClickDeleteChatpostBtn 평가
  2. 해당 이벤트 함수는, 현재 chatpost를 제거한 folders를 상태로 업데이트 -> page.tsx의 put 트리거.

3. 사이드바 Delete Folder 구현

  1. 폴더에 딸린 삭제 버튼 클릭 시, isDeleteFolderClicked 상태 토글 -> 삭제 재확인 후 삭제
  2. delete를 위한 추가 요청함수는 구현 X => page.tsx에 물려있는 useEffect중 하나가 setFolderWithPosts에 의한 변경을 감지 및 put 트리거

4. 폴더 접기, 펼치기 구현

Folder.tsx/FolderUnit 내 toggleFoldersSpread가 토글버튼 클릭에 따라, 현재 폴더 아이디에 할당된 boolean isSpread을 토글 -> Folder에서 Chatpost 렌더 조건에 걸림



3. fix : saveChatModal - chatpostName 필드명, state 변경 & category key 설정

vercel[bot] commented 1 year ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
ganoverflow ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 21, 2023 3:33pm
ganoverflow-front ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 21, 2023 3:33pm
ABizCho commented 1 year ago

확인 및 merge 부탁드려요~

ABizCho commented 1 year ago

@hongregii

  1. ㅋㅋㅋㄱ 동의합니다.. 하고나니 너무 속시원하네요
  2. 이제 대망의 main merge..!
  3. 클린코드세션 매우 필요합니다.. 아예 날잡고 리팩터링, 클린코드만 미친듯 매진해봐요 #45