Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[7주차] 기본+ 도전 과제 제출 #6

Open
wants to merge 25 commits into
base: main
Choose a base branch
from
Open

Conversation

YuSuhwa-ve
Copy link
Contributor

SERVER PR


🐕 과제 구현 명세

  • 기본과제 : 세미나 코드를 직접 쳐봄
  • 도전과제 : 이전 6주차 넷플릭스 api에 multer와 검색기능 페이지네이션 기능을 더함
    • dot env를 사용하여 .env 파일의 모든 키들을 config에 정리

image

  • getAllmedia에 페이지네이션을 적용했습니다. 이때 page, limit 둘다 없거나, 둘다 유효한 숫자로 있는 경우만 페이지네이션이 되도록 했습니다.

image

  • media 검색기능에 정렬또한 같이 구현하면서, 정렬되는 기준을 createdAt, updatedAt, createYear 3가지중 하나를 선택할 수 있도록 했습니다. 구글링 해보니 ES6부터 [ ] 를 이용하여 객체의 키부분을 변수로 구현할 수 있어 이를 사용했습니다.

image


🐥 이런 점이 새로웠어요 / 어려웠어요

  • media create api 에 express-validator를 적용하던 도중 미들웨어의 순서를 validator -> upload 순서로 했을때 validator가 제대로 작동하지 않았습니다. 구글링을 통해 두 미들웨어의 순서를 바꾸면 된다는 것을 알게되어 순서를 바꾸니 잘 동작했습니다.

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants