swsnu / swppfall2019

31 stars 22 forks source link

[HW3] POST '/api/article' follow up qs #156

Open lookdeceline opened 5 years ago

lookdeceline commented 5 years ago

155 를 보고 관련 추가 질문을 드립니다.

issue 155 에 달아주신 답변에서

{'id': ---, 'title': ---, 'content': ---, 'author': ---} 의 형태로 Response를 보내는 것이 맞습니다. 'author' 필드는 author id가 들어가는 것이 아닙니다. 'id'를 제외한 나머지 필드는 Request에서 들어온 내용을 그대로 Response로 되돌려 보내주시면 됩니다.

라고 하셨는데,

과제 스펙을 보면 article 과 관련된 모든 post / put request 는

Article : {title: string, content: string}

형식으로 이루어진다고 명시되어있습니다.

그렇다면 {'id': ---, 'title': ---, 'content': ---} 의 형태로 reponse를 보내야 하는 것 아닌가요?

감사합니다.

kyunggeunlee commented 5 years ago

죄송합니다 #155 에 틀린 답변을 해서 방금 수정했습니다. {'id': ---, 'title': ---, 'content': ---, 'author': ---}의 'author'에는 author id가 들어가는 것이 맞습니다.

lookdeceline commented 5 years ago

@kyunggeun-lee
그렇다면 정정해주신 답변에서

'id'를 제외한 나머지 필드는 Request에서 들어온 내용을 그대로 Response로 되돌려 보내주시면 됩니다.

이 부분에 따르면, Request로 들어온 내용은 {title: string, content: string} 이므로 'author' 항목은 없기 때문에 Response가 {'id': ---, 'title': ---, 'content': ---} 이 되야 할 것같은데, 이부분은 무시하고

{ 'id': article id, 
'title': Request로 들어온 'title' value, 
'content': Request로 들어온 'content' value, 
'author': article author id }

형태로 Response를 보내면 되는 것인가요 ??

아니면

'id'를 제외한 나머지 필드는 Request에서 들어온 내용을 그대로 Response로 되돌려 보내주시면 됩니다.

에 따라서

{ 'id': article id, 
'title': Request로 들어온 'title' value, 
'content': Request로 들어온 'content' value, 
}

형태로 Response를 보내면 되는 것인가요 ?? 감사합니다!

ktaebum commented 5 years ago

@lookdeceline

{ 'id': article id, 
'title': Request로 들어온 'title' value, 
'content': Request로 들어온 'content' value, 
'author': article author id }

로 HttpResponse content에 들어가면 됩니다

lookdeceline commented 5 years ago

감사합니다 !

coiger commented 5 years ago

조교님, HttpResponse content에 들어가야 한다고 하셨는데, 해보니까 잘 안됩니다. 혹시 JsonResponse 일까요?

ktaebum commented 5 years ago

@kanghc0217 음 아니요 HttpResponse에도 content를 넣을 수 있습니다 dump된 json을 넣으시면 됩니다

tho-kn commented 5 years ago

JsonResponse로 구현해도 상관은 없는거죠? 그리고 article과 유사하게 comment를 post하거나 put할때도 content만이 아니라 article과 author에 각각의 id를 같이 보내줘야하나요?

ktaebum commented 5 years ago

@th-snu

  1. 네 JsonResponse가 HttpResponse의 subclass라서 상관 없습니다.
  2. 네 그렇게 해주시면 됩니다.