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

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

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

Conversation

YuSuhwa-ve
Copy link
Contributor

SERVER PR


🐕 과제 구현 명세

  • 공통 : DTO와 constants들을 사용해 좀더 깔끔하게 코드를 리펙토링

  • 기본과제 : 세미나 시간의 코드를 따라치고 나머지 read, update, delete 기능을 리팩토링함

    • 조회, 업데이트, 삭제 기능에 auth를 넣어 인증된 사용자만 사용가능하도록함
      • 이때 개별조회, 업데이트, 삭제 기능은 조회-업데이트-삭제 하고자하는 유저만 해당 기능을 사용 가능하도록 함

image

- (솝커톤때 팟장님한테 배운) DTO와 스프레드 연산자를 활용해 update를  범용적으로 사용가능

image

image

  • 도전과제 : 기존 media crud로 구성된 넷플릭스 api에 회원가입, 로그인 기능을 추가함

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

  • 어려웠던점 : update 기능을 리팩토링하며 body에 필드가 있을 경우만 유효성 검사를 하고 싶었는데, validator의 메소드들은 해당 필드자체가 없으면 error을 생성해서, 유효성 검사를 따로 하였음... 추가로 공부해서 기존 메소드에 있는지 아니면 custom으로 만들어야하는지 봐야할듯

image

  • 멋쟁이 준서님이 올려준 아티클을 읽고 기존 rds 에 새로운 데이터베이스를 추가해 써보았음. 아주 편하군요!

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