-
Notifications
You must be signed in to change notification settings - Fork 4
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
Express 김세환 sprint6 #2
base: express-김세환
Are you sure you want to change the base?
The head ref may contain hidden characters: "express-\uAE40\uC138\uD658-sprint6"
Express 김세환 sprint6 #2
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
세환님 고생하셨습니다!
mock, seed 같은걸 보니 따로 학습을 열심히 하시는 것 같아서 보기 좋습니다 ㅎㅎ
전반적으로 깔끔하게 작성해주셔서 몇가지 코멘트만 드립니다
수고하셨습니다~
|
||
app.use(cors()); | ||
|
||
const dbUser = process.env.DB_USER; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
요기부터 아래쪽까지 앞에 빈공간이 있습니다. 요런건 지난번에 말씀드린 prettier를 적용해주시면 깔끔해집니다~
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
목업데이터를 따로 추가해주시다니 좋네요 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
totalCount 는 필요없는것 같고, list에 안넣어놓고 data에 그대로 넣어놔도 될것 같아요
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
seed까지 추가해주시다니 좋습니다 ㅎㅎ
const ProductSchema = new mongoose.Schema( | ||
{ | ||
name: { | ||
type: String, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
validation 같은것들도 추가해보면 좋을것 같아요~ min, max 등이요!
요구사항
기본 요구사항
공통
백엔드 구현 요구사항
중고마켓
Product 스키마를 작성해 주세요.
상품 등록 API를 만들어 주세요.
상품 상세 조회 API를 만들어 주세요.
상품 수정 API를 만들어 주세요.
상품 삭제 API를 만들어 주세요.
상품 목록 조회 API를 만들어 주세요.
각 API에 적절한 에러 처리를 해 주세요.
각 API 응답에 적절한 상태 코드를 리턴하도록 해 주세요.
.env 파일에 환경 변수를 설정해 주세요.
CORS를 설정해 주세요.
render.com로 배포해 주세요.
MongoDB를 활용해 주세요.
주요 변경사항
스크린샷
멘토에게