codestates / codestates/Naengttatouille

[KPT] 이종석 2021-10-15

Open
#113 0 comments 2 reactions 4 assignees Claimed by @JIYEONGYANGdev View on GitHub
Dominant language
JavaScript
Stars
0
Forks
0
PR merge metrics
No merged PRs in 30d

Description

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

### Keep (유지할 항목)
* 클라이언트에서 서버로의 요청을 분기하여 Routes, Controllers 폴더를 생성하고 목적에 맞게 하위 폴더를 생성 및 정리하여 흐름을 확실히 잡고 가는 점이 좋았습니다.
* 클라이언트와 연동하면서 진행하다 보니 CORS 요청의 목적에 맞는 속성, COOKIE와 토큰에 대해 적절한 속성이 무엇인지를 알 수 있었던 점이 좋았습니다.
* Sequelize에서 models,migrations,config,seeders 각각의 폴더가 정확히 어떤 역할을 하는지 하나하나 짚고 이해를 할 수 있었던 점이 좋았습니다.

### Problem (문제라고 생각하는 항목)
* [client] 로그인 화면, 회원가입 화면에서 인풋창을 컴포넌트로 구성하여 map을 돌렸기 때문에 위치배정을 일일히 할 수 없어 불편함을 느껴, 다음 프로젝트에서 기획을 짤 때 잘 논의해야할 것 같습니다.
* YouTube API 이용 시, 쿠키 사용을 위한 cors 설정 credentials: true, axios.defaults.withCredentials=true 옵션과 충돌이 일어나 로그인 여부와의 결부가 해체되어야 영상을 받아와 렌더할 수 있는 문제가 있다. 에러를 검색해보고, fetch 를 이용해보는 방법도 찾아보기로 했다.
* [Back] AWS를 통한 배포 자동화 과정에서, aws parameter store를 이용해 환경변수를 export해 사용하려 했으나, 쉘 스크립트가 작동하지 않아 dotenv를 사용하는 것으로 만족해야 해서 아쉬웠다.
* git을 잘 활용하지를 못 했었어서, merge를 하지 못하고 새로운 기능을 구현할 때마다 branch만 계속 쌓여갔고, git을 다시 이해하려 노력해야겠다.

### Try (Action Items)
* [Client] 컴포넌트로 만들어야할 것과 만들지 않아야할 것의 리스트를 만들어야 할 것 같다
* [Back] 서버를 HTTP로 만들었기 때문에 HTTPS로도 시도해봐야 할 것 같다
* [Back] Typescript에 대해 배워서 사용할 것이다
* [Back] TypeORM을 생각해보고 있다
* [Back] Nest.js에 대해 배워볼 것이다

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.