codestates / codestates/camvillage-client

[KPT] 둘째주 월요일 KPT 회고

Open
#21 0 comments 2 reactions 1 assignee Claimed by @sparklingwater226 View on GitHub
KPT
Dominant language
JavaScript
Stars
0
Forks
0
PR merge metrics
No merged PRs in 30d

Description

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

## Keep (유지할 항목)
- 평일이나 주말에도 디스코드를 통한 [현재 진행완료된 작업내용 / 주말동안 남은 to-do]공유가 좋았음
- 에러사항이 생겼을 때 바로바로 톡방에 공유
- 팀원 전부 맡은 바 임무를 최대한 완수하려고 노력하고 있으며 활발한 소통으로 잘 진행되고 있음

## Problem (문제라고 생각하는 항목)
- [front] search post 요청 데이터를 일반 body parameter가 아닌 url query parameter로 보내는 게 맞을 것 같음 (현재 검색 버튼을 누를 때만 body에 카테고리 및 검색어 담아 보내도록 구현되어 있는데 이 경우 검색→검색결과 조회→타 페이지 방문→뒤로가기 눌러서 검색결과 페이지로 돌아감 이 flow에서 마지막에 검색 결과가 제대로 렌더되지 않고 있음)
- [back] db 에서의 잦은 스키마 변경으로 모델과 마이그레이션 파일의 잦은 수정으로 인해서인지 db쪽 에러가 많은 것 같음.
- [back] pr 및 merge를 할 일이 생각보다 많았음. 특히 서버에서 스키마 변경이 일어날때마다 해당 코드들이 공유되지 않은 팀원은 달라진 스키마 및 레거시가 된 코드로 인해 에러나 충돌이 일어남.

## Try (Action Items)
- [ ] 테스트시 팀원 전원이 동작을 확인하고 오작동 및 에러 발생시 해결방안 모색
- [ ] [front] search post 요청 수정 및 태그 사용하도록 수정
- [ ] [front] css 개발 착수
- [ ] [back] 마이그레이션과 같은 스키마 변경이 일어날 때 해당 사항을 가능한 빨리 pr하고 해당 사실을 다른 팀원에게 알려 merge하도록 하기

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.