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 3 commits into
base: main
Choose a base branch
from
Open

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

wants to merge 3 commits into from

Conversation

m1njae
Copy link
Member

@m1njae m1njae commented Nov 30, 2022

SERVER PR


🐕 과제 구현 명세

  • [기본] 오늘 세미나에서 나온 코드들을 직접 타이핑하며 익혀보기
  • [도전] 넷플릭스 API에 API 추가
    content 검색, pagination 기능을 제공하는 API를 추가하였고, content 등록 API에는 이미지 다중 업로드로 구현을 해보았습니다.

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

  • search와 option 그리고 pagination이 제법 복잡한 기능이라고 생각했는데 왜 prisma가 쉽고 트랜디한지 몸소 깨닫게 되었던 세미나였습니다. 너무나도 간편하게 기능들을 제공해주는 것 같아요!
  • content 등록 API를 구현하면서 Form-Data는 값을 문자열을 자동변환해준다는 사실을 알게되었습니다! 그래서 ageLimit 속성을 형변환하는 과정을 거쳤습니다.
  • 기능을 구현하다가 문득 궁금해진 사실인데, content 정보를 등록 시 이미지는 따로 table을 두어서 관리를 하는지, 아니면 content table에 column으로 넣어서 관리하는지에 대한 고민이 생기게 되었습니다 ! 혹시 권장하는 방법이 있거나 이 방법이 좋을 것 같다라는 의견이 있으신 분들은 공유 부탁드려요😊

Comment on lines +10 to +13
const images: Express.MulterS3.File[] = req.files as Express.MulterS3.File[];
const imageList = images.map((image: Express.MulterS3.File) => {
return image.location;
})
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

이미지 여러 개를 올릴 때는 [] 하면 되는군요!!👍

router.get('/search', contentController.searchContentByName);

//* 콘텐츠 생성 POST api/content
router.post("/", upload.array("file"), contentController.createContent);
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

single 대신 array를 사용하면 되는군요 굿굿

@onpyeong
Copy link

onpyeong commented Dec 1, 2022

도전과제까지 하느라 고생 많았어유!! 이미지 저장은 저도 궁금하네요!

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

Successfully merging this pull request may close these issues.

2 participants