codestates / codestates/Maat-Bab

[KPT 회고] 2021.11.05

Open
#193 1 comment 2 reactions 4 assignees Claimed by @js5787 View on GitHub
retrospect
Dominant language
JavaScript
Stars
0
Forks
1
PR merge metrics
No merged PRs in 30d

Description

### Keep (유지할 항목)
* 잘 풀리지 않는 부분을 팀이 합심해서 해결한 덕분에 길게 끌지 않고 해결할 수 있었다.
* 문제가 생겼을 때 해당 기능 담당 팀원에게 상의하며 빠르게 문제를 해결했다.
* 각자 컨디션에 맞춰 아침 코딩 시작을 정하고 인터넷 상태에 따라 소통 방법을 달리하는 등의 유연성이 팀 컨디션 향상에 도움이 된 거 같다.
* 팀장, 팀원 모두 역할에 맞게 자신의 업무에 최선을 다하고 있다는 생각이 들어 보기 좋았다.
* 문제가 생겼을 때 서로의 코드를 보고 여쭤보고 수정하는 모습을 계속 유지해갔으면 좋겠다.

### Problem (문제라고 생각하는 항목)
* 프론트와 백 모두 기능적으로 구현해야 할 것이 당장은 없고, 프론트의 CSS만 남았다.
* 처음에는 기능적인 부분에 대한 생각만 중심적으로 고민하고 있었지만, 놓칠만한 보안적인 부분을 더 생각해봐야할 것 같고 기능적인 부분에 있어서는 잘 구현이 되었지만 필요 없는 코드가 있는지 간결하지 않은지 정리해봐야할 것 같다.
* 11/4일자로 백엔드의 경우 필요한 기능 구현을 완료하였고, 프론트엔드의 경우에도 채팅 참여자의 음식취향, 식사예절 프로필을 보여주는 mateList 컴포넌트 등 정도만 남았다. 모바일 대응 등 반응형을 포함한 CSS의 완성도가 아직 미숙하여 더 작업해야 할 것이다.
* bare-minimum과 advanced 기능들을 필수적인 요소들부터 수립해서 목표한 기능은 완성할 수 있었던 것 같은데, 논의해 두었던 추가적인 기능들을 고려해야할 것 같다.
* [프론트] 완성도를 높이기 위해 수정 및 보완할 부분들은 다음과 같다.
* 모달컴포넌트의 경우 모달 바깥을 누르면 꺼지도록 수정해야 한다.
* Main 페이지에서 약속을 조회하기 위해 '지역(시)', '지역(구)', '음식점', '날짜' 등의 option 및 value를 선택 후 약속만들기를 클릭하면, 해당 값들이 내려지도록 구현해야 한다.
* 현재 약속카드 조회 시 날짜 쿼리 기본값(현재 날짜)이 쿼리로 담겨져 요청을 보내는 상태이다. 기본값이 없도록 조정해야 한다. (당일만이 아닌 여러 일자에 해당하는 약속 카드를 모두 조회하기 위함)
*[백] Chat 페이지에서 대화에 참여하고 있는 상대방의 프로필을 mateList에서 렌더하기 위해 다른 참여자의 user_id가 필요했다. 따라서 기존에 작성한 card 조회 엔드포인트 에서 card_id 쿼리만을 보내어 참여한 user_id 와 host(카드를 생성한 방장) 여부 등까지 포함해서 응답을 보내는 것으로 수정하였다.

### Try (Action Items)
* 남은 시간과 프론트의 작업량 등을 고려해서 추가할 기능을 정해야겠다.
* 예를 들어
- 현재 채팅방에서 2인 뿐만이 아니라 다인원의 실시간 대화가 가능하므로, 여러명이 참여할 수 있는 기능임을 보여주는 쪽으로 수정해야하는 것(약속카드 만들기 에서 인원 수 option 값 2인 고정 해제
- 카드를 만든 '방장' 권한(기능) 추가, 만든 약속 카드 자체를 '삭제'할 수 있는 기능(다른 참여자가 있다면 방장이 위임되는 기능)
- 현재는 '나가기' 기능이 있으며, 개별 참여자가 각각 나가 모두 나가면 카드가 삭제되는 것으로 구현하였다.
- 채팅 알림 기능이 필요할 것이다. 유저가 참여하거나, 메시지를 보낸 경우에 대한 알림 기능이 필요하다고 생각된다. (어느 페이지에서 알림을 뜨게할 것인지, 로그인 후 메인페이지 등 채팅 페이지가 아닌 곳에서도 뜨게할 것인지 고려해봐야할 것이다.)
- 추가적으로 현재 서울시를 대상 지역으로 구현하였는데, 전국적으로 가능한지 nightmare 로 고려해봐야겠다.

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.