codestates / codestates/Naengttatouille

[KPT] 2021-10-14

Open
#81 1 comment 2 reactions 4 assignees View on GitHub

@js5787 is already working on this.

Since Oct 13, 2021.

Dominant language
JavaScript
Stars
0
Forks
0
PR merge metrics
No merged PRs in 30d

Description

팀원 중 한 분이 작성해주세요.
Keep, Problem 항목에 대해서 매주 정기적인 날에 회고를 진행하여 기록하고, 그것을 기반으로 Try (Action Items) 이슈를 생성해주세요.

Keep (유지할 항목)
  • 회원가입, 로그인 뿐만 아니라 Main 컴포넌트 관련 로직에서도 클라이언트-서버 간 실시간으로 코드를 리뷰하여 연동해볼 수 있었던 점이 좋았다.
  • 팀원 간 로컬에서 DB를 sequelize 마이그레이션 등을 통해 로직과 연동해보던 것을 rds 인스턴스로 옮겨(환경변수 연결) 번거로움을 덜었던 점이 좋은 것 같다.
Problem (문제라고 생각하는 항목)
  • [Client] 로그인 상태여부와 유저 냉장고 로드가 결부되어 있어 새로고침이나 로그아웃 시 유저의 냉장고 식재료 목록이 가져와지는 로직이 조금 복잡해졌던 것 같다.
  • YouTube API 이용 시, 쿠키 사용을 위한 cors 설정 credentials: true, axios.defaults.withCredentials=true 옵션과 충돌이 일어나 로그인 여부와의 결부가 해체되어야 영상을 받아와 렌더할 수 있는 문제가 있다. 에러를 검색해보고, fetch 를 이용해보는 방법도 찾아보기로 했다.
  • [Back] AWS를 통한 배포 자동화 과정에서, aws parameter store를 이용해 환경변수를 export해 사용하려 했으나, 쉘 스크립트가 작동하지 않아 dotenv를 사용하는 것으로 만족해야 해서 아쉬웠다.
Try (Action Items)
  • [Client] CSS를 마무리해야할 것이다. 와이어프레임에서 기획한 목업대로 div 박스로 자리를 잡아두느라 일일이 렌더하던 것을 map으로 처리하면 좋을 것 같다.
  • [Client] 사용자와 admin 권한이 구분되는 기능(기본 식재료 목록 추가, 삭제 권한 등)을 마무리해야 할 것이다.
  • [Client] CSS를 상위 컴포넌트에서부터 내려오며 수정해야한다. 최상위 컴포넌트에서부터 하위로 내려와야 정렬이 가능할 것이다. 필요하다면 세부 컴포넌트로 분리, 추가할 수 있을 것 같다.
  • [Back] 클라이언트 배포 마무리를 위해 CodeBuild 에서 빌드프로젝트의 IAM role을 수정해야 할 것이다.
  • 발표 준비를 위해 시연을 시범해보고, 기능 및 서비스에 대한 고민이 담긴 내용으로 스크립트를 준비해봐야할 것이다.

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.