codestates / codestates/Naengttatouille
[KPT] 2021-10-14
Open
@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
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Assessment
This issue has not been assessed yet.