팀 회고 Week3

🤜지난 주 도전할 점. 잘 수행하셨나요?

• 주말 동안에 다른 분들이 작성하신 코드를 뜯어보면서 이해해보자.(세연)
  • 근데 이번주에 또 쌓임 ㅋㅋ 주말에 또 공부
• 저희도 git hook을 도입해서 푸시할 때 타입스크립트 체크를 자동화하는 것도 좋을듯? (기종)
  • 대신 휴먼 타입스크립트 했습니다 push하기 전에 일일히 tsc 돌리고 lint 돌리고
• 기술적인 고민 페이지를 더 활용해 보자. 공유된 페이지로 새로 파겠습니다 (기종)
• 다른 분들이 작성한 코드 클론코딩해보기, 2일차에 빠지다보니 2일차에 진행한 부분들은 헷갈리는 경우가 많았음. ( 종찬 )
• 반성합니다… (기종)
• 반성합니다2…(종찬)

😀좋았던 점

• 내가 구현 할 수 있는 선 안에서 최대한 예쁘게 시각화를 해보려 했는데, 구현하고 보니 결과가 꽤 이쁜것 같아 만족(세연)
• 팀원분들이 다 열정적으로 개발에 참여해주시고 피드백도 엄청 빠르고 리뷰도 빠르게 받을 수 있어서 좋았다 (정욱)
• 팀 회고 시간에 처음으로 도입했던 노션 템플릿. 4주차에도 활용할 계획입니다. (기종)
• 저희 프로젝트로 제 노션페이지를 담아보니까 엄청난 감동이 몰려왔던 것 같습니다 ( 종찬 )

😖부족한 점

• merge conflict를 해결한다고 github의 conflict 해결 페이지 안에서 눈으로 보고 해결 후 merge를 했는데 실제로 빌드를 돌려보니 와장창 터졌다.. merge conflict 나면 꼭 해결 후 빌드도 잘 되는지 확인해보자..!(세연)
• import 컨벤션을 맞춰줘야 할 것 같다 (기종)

import { useState } from "react";
import { useThree } from "@react-three/fiber";
import MyComponent from "./component/MyComponent";
import { between } from "../utils/utils";

• 순서 관련 린트가 있군요 (하지만 린트가 저희가 정한 것과 다르게 동작한다면?-컴퓨터 못믿음)
  • 외부 모듈
  • 컴포넌트
  • 훅
  • 유틸
  • 스타일
  • 타입
  • 애셋
• 현재 진행중인 테스크 및 완료된 테스크에 대한 확인, 문서화가 흩어져 있어서 통일을 하고 한곳에서 모아서 보여줘야 할 필요가 있다 (정욱)
• 기술적인 고민 기록을 안 썼습니다… 활성화가 안 되어 있다는 점이 아쉬웠음 (기종)
  • 개인적으로는 귀차늠… 맨날 새벽까지 구현만 작업하는데 시간이 어딨음
  • 기술적인 고민보다는 구현을 어떻게하지??? 이 상태가 더 많았던 것 같습니다
    • 이런것도 써주세요 중요합니다 매우 저희의 코드의 의식의 흐름을 알 수 있으니까
  • PR에 기술적인 고민도 다 써놓는 건 어떠신가요
    • 가장 구현한 직후니까 쓰기 편하고 어차피 글을 쓰는 시간이라
    • 노션은 나중에 옮기시고
• 목요일 태스크맵을 잘못짜서 백엔드 API 개발하는데 혼선이 생김
  • 태스크맵… 월요일에 다같이 논의하고 고칠 건 고치고 뺄 건 뺍시다
    • 기본적인 얼개만 잡아놓을게요 (기종)

🤜도전할 점

• 진행중인 태스크, 완료된 태스크를 볼 수 있는 공간을 마련해두자 ex) 노션에 진행상황 페이지 만들기, github project에 sprint 안에 이슈를 생성해서 관리(세연)
• PR 쓰는 겸 기술적인 고민/설계 고민 등 코딩하면서 들었던 생각들을 남겨주세요 (기종)
  • 진짜 거창하지 않아도 좋습니다
  • 노션으로는 나중에 옮깁시다
• 여러분들 쉽시다 새벽까지 일하면 힘들어요 (기종)
  • 목요일 개인 개발 완료 시각을 오후 7시로 잡고 9시에 딱 2시간만 배포 만지고 자러가셔야죠
    • 여러분의 수면시간을 보장합시다

☔개선사항 드래프트! 자유롭게 써 주세요

여러분들이 한 주를 회고하면 아쉬운 게 많아서 폭주를 하게 되므로 해당 부분을 마련했습니다 -기종

프로덕트 개선사항

• 현재는 방향에 따라 같은 키워드를 갖는 문서를 분류하고 있는데, 해당 키워드가 쓰인 이정표 같은 게 있으면 무슨 공통 키워드인지 알 수 있어서 좋을 것 같다

• 짧은 소제목의 경우는 굳이 애니메이션이 들어가지 않아도 무방하므로 해당 기능을 제거한다.

• 로딩 프로그레스 바 구현

• hidden을 활용해서 너무 멀리 있는 것들은 scene에서 렌더링을 막아버리자!

• 공개된 노션 페이지의 경우 링크를 추가하는 것도 좋을 것 같다

• 루트페이지를 1개만 만들어볼까요

  • 서브페이지가 없으면 휑한 화면이 보여질텐데 너무 DB형 노션에만 비주얼이 최적화있는듯

• misc(어느 키워드에도 해당하지 않는 페이지)의 경우 고립된 섬으로 만드는 것도 좋았을듯?

• 미들웨어는 (redis에 부담이 될 수 있어서) 어떤 건 쓰고 어떤 건 안 쓰도록 구현하고 있는데, 새로 미들웨어를 쓰는 api를 추가하려면 index.html에서도 수정해줘야 해서 부담스럽다.

  import {authMiddleware,catchAuthError} from "../middleware/authMiddleware";
  
  router.post(
    "/gallery",
  	authMiddleware,
  	catchAuthError,
    asyncHandler(async (req, res) => {
      //duration= 2w||1m||3m||1y
      const userID = req.userid;
      const notionAccessToken = req.accessToken;
      const nowTime = Date.now();
      const { period = "all", theme = "dream" } = req.query;
  
      const notionRawContent = await getRawContentsFromNotion(notionAccessToken, period);
      const processedNotionContent = await processDataFromRawContent(notionRawContent, theme);
      const galleryID = await saveGallery(userID, processedNotionContent);
  
      console.log(`총 처리 시간: ${Date.now() - nowTime}`);
      res.status(200).json({ page: `/gallery/${userID}/${galleryID}` });
    }),
  );

  이런 식으로 수정하는 게 괜찮을듯. 명시적으로 이 엔드포인트가 무슨 선택적인 미들웨어를 쓰는지 알 수 있어서 응집성이 늘어남

• TP(TelePort)를 넣어보아요(0,0으로 리스폰)

  • 캔버스 밖 컴포넌트가 관여되므로 (캔버스 밖+캔버스 안 상호작용)
  • 플레이어 위치도 전역 상태로 관리해야하나…

협업 스타일 개선사항

• PR 템플릿이 처음부터 뜨게 하는 게 필요해요…
• import 컨벤션이 필요해요…
• 아무래도 기술적인 문제들이 문서화가 잘 안 되는 것 같습니다
  • Web17 도지 하나 도지 둘이 괜찮아 보이는데 그거 차용하는 것도
  • 개인 기술적 문제는 PR 메시지 적는 겸 기술적인 문제들을 적으면 나중에 안 귀찮지 않을까요
• API 문서화 필요하나요 여러분
  • 다른 사람들에게 프로젝트 소개할 때 필요하기도 할듯