codestates / codestates/art-ground

[Dev Log] 20211001/ 문순려

Open
#274 0 comments 0 reactions 1 assignee Claimed by @Sunryeo View on GitHub
Dev Log
Dominant language
JavaScript
Stars
7
Forks
9
PR merge metrics
No merged PRs in 30d

Description

### **오늘은 어떻게 프로젝트에 기여했나요?**

- admin.ctrl.js getAllReviews 함수 테이블 데이터 엮어서 응답 돌려주기 리팩토링 및 에러 픽싱
- mypage/likes.js getMyLikes 함수 테이블 데이터 엮어서 응답 돌려주기 리팩토링 및 에러 픽싱

### **오늘의 프로젝트에서 힘든 점은 무엇인가요?**

- 시퀄라이즈 findAll 메소드 옵션에서 include.model 옵션을 사용하면 관련된 테이블의 정보를 함께 받아올 수 있다는 사실을 확인하고 코드 리팩토링을 무작정 하다 보니 관련되어 있지 않은(not associated) 테이블은 정보를 받아올 수 없다는 사실까진 확인하지 못해서 결국 코드가 원점으로 돌아와버렸다. 한 큐에 전부 받아올 생각을 했으니.. 당연히 안될 수밖에 없다. 결국 반복문을 다시 쓰게 되었지만 관련된 테이블 하나는 처음 받아올 때 include로 원하는 필드의 정보를 받아올 수 있었다. 그래도 새롭게 시퀄라이즈 메소드 문법을 터득하게 되어 좋은 경험이었다고 생각한다.
- 생각보다 코드 리팩토링이 꽤 시간투자를 해야 하는 작업이라는 생각이 든다. 뭐든 마무리가 가장 어렵다고 하던데 잊지 말자, 끝날 때까지 끝난 게 아니다! 끝이 다가올수록 해이해지려 하는 마음을 붙잡고 마지막까지 조금만 더 꼼꼼히 체크해야겠다.

### **내일은 프로젝트에 기여하기 위해 무엇을 해야 하나요?**

- [ ] controllers 디렉토리 전체 라우팅 코드 리팩토링
- [ ] 정제된 더미 데이터 찾기
- [ ] about 페이지에 삽입될 텍스트 작성하기
- [ ] 배포환경에서 카카오 로그인 기능 테스트
- [ ] 시스템 아키텍쳐 다이어그램 설계

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.