[FE] [team-31] 2주차 2번 째 PR 요청드립니다. #237
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- 빈 객체 검사
- 작성용 카드 템플릿 추가 기능 구현
- 카드 취소 버튼 클릭 시 writable 카드 삭제 - 카드 등록 버튼 클릭 시 POST 기능 추가 - 템플릿 추가 함수와 중복 되는 부분 합침 - 카드 추가 버튼 disabled 속성 토글하는 함수 추가 - 입력 유효성 검사 - 사용하지 않는 util함수 삭제
- popup 컴포넌트 추가 - 함수 분리
- 필요없는 css 폴더 삭제
수정 카드 나타날 시, 카드 클래스에 따라 버튼 색 바뀌도록 수정
- 함수 이름 변경
- css 오류 수정 - writable 카드 템플릿 카드 수정 기능에서도 사용할 수 있도록 수정
- controller의 상태를 board에게 넘겨줌 - 카드 이벤트들이 controller에서 board로 이동 - 팝업 이벤트도 popup으로 이동
- controller의 상태를 board에게 넘겨줌 - 카드 이벤트들이 controller에서 board로 이동
…rs-2022#154) * feat : 테이블 인덱스 컬럼 추가 closes #15 * refactor: [BE] 1주차 금요일 pr 피드백 적용 closes #16 * feat : 로그로직 작성 - 로그 조회 기능 구현 - 로그 저장 기능 구현 closes #19 * feat: 카드 Status별로 조회 기능 구현 Closes #23 * feat: CardListDto 생성하고 관련 로직 수정 Closes #29 * refactor: 카드 생성 로직 변경 - addDto에 인덱스값과 스테이터스값 같이 받아서 생성 closes #31 * test: CardRepositoryTest 작성 * test: 테스트 작성하기 - 카드 컨트롤러와 리퍼지토리 테스트 하기 closes #21 * Build: build.gradle, application.properties 수정 - mysql 의존성 추가 - yml 파일 생성 Co-authored-by: leekm0310 <[email protected]>
- 여러 기능을 한꺼번에 하던 함수 쪼개기 - 네이밍 좀 더 구체적으로 개선
- BoardViewModel, popup에도 private 속성 추가
- fetch error 처리 try..catch 적용
- header 높이 제외하고 height 100vh 던 부분 삭제 - size.scss 삭제
- 이벤트 대상 main으로 변경 - main에 insertAdjacentHTML 에서 innerHTML로 변경
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
안녕하세요!
저희는 team-31 에 dony, 포키 조입니다.
리뷰 반영해서 보내고 싶어 PR이 많이 늦었습니다..ㅠㅠ
지난 번 리뷰 때 코드 정리를 못하여 많이 복잡했을텐데 꼼꼼히 리뷰 달아주셔서 감사합니다!
2주동안 정말 많이 도움됐고 마지막 PR도 잘부탁드립니다!
지난 리뷰를 실수로 조원분이 close하여 지난 리뷰때 코드가 겹치는 커밋들이 있습니다.
전체적으로 고쳐 새로 보셔도 되나 불편하실거 같아 이번에 수정한 커밋 내역들만 남겨놓겠습니다.ㅠㅠ
4/15 에 0cdc2da 부터 끝까지 보시면 됩니다!
구현 결과 ✅
2022-04-18.12.15.47.mov
리팩토링 🛠
이번에는 리팩토링과 실제로 DB에 데이터를 받아오는 작업에 중점을 둬서 했습니다.
카드 수정과 삭제, 추가 이렇게 나누어 맡아 진행하다보니 후에 코드를 합치느라 코드가 많이 복잡해지고,
비슷한 기능에 다른 소스들을 합치느라 시간이 오래걸렸습니다.
나름 많이 토론했다고 생각했는데 비슷한 기능이 겹친다는 걸 빠르게 파악 못해 아쉽습니다.
지난번 리뷰때 말씀하신 만능 기능을 하는 함수가 아닌 한가지 기능만 하도록 쪼개보았습니다!
생각해보니 저도 toggle 기능을 하는 함수들이 안에서 값이 어떻게 바뀌어 결과를 이렇게 반환했는지 파악하는데 힘들었다는 생각이 들었습니다.
때문에 이 부분을 가장 중점적으로 나누어 코드 흐름이 잘보이도록 더 신경써봤습니다.
이번 프로젝트에서 느낀 점 🤔
백엔드 영역도 더 알아야겠구나 생각했습니다.
API를 받더라도 fetch 후 에러가 떴을 때 코드를 몰라 저희 잘못인지 아닌 지 구분 못할 때가 있었습니다.
또, 저희가 백엔드 영역을 모르는만큼 그 분들도 저희 영역을 잘 모르기 때문에 데이터를 어떤 식으로 만드는게 좋을 지 토론을 많이 하고 요청도 많이 해야겠다고 느꼈습니다.
당연하게 제가 생각한 형태의 데이터로 주실거라 생각했던 것 같습니다.
이번 프로젝트에서 기능 구현을 많이 못해 너무 아쉽지만,
첫 플젝을 망할 수 밖에 없다는 크롱의 말처럼 협업할 떄 소통 방식과 깃에서 작업 방식들을 익혔다고 생각하며 위로하고 있습니다 ^^;
궁금한 점 🧐
사실 이전에 여러기능을 하는 함수들을 만든건 중복 코드를 없애려다보니 나온 상황이기도 한데.. 중복 코드가 좀 생기더라도 지금처럼 한가지 기능만 하도록 하는게 더 좋을까요?
확실히 나누다보니 흐름이 잘보이게 되서 좋지만 이전에 비해 코드량 늘어난게 보이기도 해서 질문 드립니다!