codestates / codestates/Naengttatouille

[KPT] 이선웅 2021-10-15

Open
#111 0 comments 3 reactions 1 assignee View on GitHub

@seonwoong2 is already working on this.

Since Oct 15, 2021.

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

Description

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

Keep (유지할 항목)
  • client작업을 할때 같은 프론트엔드 팀원분과 커뮤니케이션을 자주해서 머지할 때 충돌이 일어나지 않았다 작업할 때 실시간으로 커뮤니케이 필요하다고 느꼈다.
  • 게터타운을 이용해서 팀원간에 의사소통이 원할하게 이루어져서 필요한 정보나 이슈가 생겼을 때 빠르게 공유할 수 있어서 좋았다
Problem (문제라고 생각하는 항목)
  • 재료목록을 유저가 볼 수 있도록 뿌려주는부분을 2차원배열?형식으로 뿌려주려고 하였는데 css를 이용하지 않고 함수를 이용해서 뿌려주었는데 너무 비효율적 이라고 생각이 든다
  • 목록을 추가하거나 삭제할 때 window.location.reload()함수를 이용하여서 새로고침해서 업데이트 된 상태를 보여주는데 각데이터의 상태가 변할때 목록을 새로 가져올 수 있도록 좀 더 공부를 해봐야할 것 같다.
Try (Action Items)

*CSS를 통해 해결할 수 있는 부분을 react로 어렵게 로직을 구현해서 힘들게 돌아간 느낌이 크게 들어서 CSS에 대해 좀더 깊게 알아봐야할 필요성을 느꼈다

  • 이번 프로젝트는 시간이 짧아서 준비를 할때 자세하게 정리를 하지못하고 시작하였는데 다음 프로젝트때는 좀더 꼼꼼하게 준비하고 시작해야된다고 생각하였다
  • 다음 프로젝트에서는 이번에 구현하지 못 하였던 기능들을 구현해 봐야겠다.

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.