wafflestudio/seminar-2020

(옮김) SurveyResultSerializer의 os_name Field에 대하여

Open

#162 opened on Sep 15, 2020

View on GitHub
 (1 comment) (0 reactions) (0 assignees)Python (49 forks)auto 404
HW2backendgood first issuequestion

Repository metrics

Stars
 (27 stars)
PR merge metrics
 (PR metrics pending)

Description

과제 2 기반 서버 코드를 grace day 3일을 고려한 9/14(월) 이전에 https://github.com/davin111/waffle-rookies-18.5-backend-1 에 공유해두었었는데, 해당 repository에 올라왔던 Issue가 있어 여기에 공유합니다. 여기에서 질문해주세요. 아래는 옮긴 내용입니다.

SurveyResult 모델에 os_name이라는 항목이 없는데 SurveyResultSerializer의 Meta의 fields가 'os_name'을 포함하고 있습니다. SurveyResult에 os라는 ForeignKey가 있으면 os_name이라는 field도 Serializer가 자동으로 연결해 주는 것인가요?

SurveyResultSerializer에서

os_name = serializers.CharField(write_only=True)

정의 부분을 삭제하고 SurveyResultSerializer를 생성해 보니 django.core.exceptions.ImproperlyConfigured: Field name `os_name` is not valid for model `SurveyResult`. 가 발생하는 것을 확인할 수 있었습니다. 따라서 os_name을 자동으로 연결해 주지는 않는다고 생각했습니다.

그렇다면 #1 처럼(해당 private repository에서 이 line을 삭제하는 PR이 올라왔었습니다.) os_name을 Meta에서 삭제하는 것이 적절한 방법일까요?

Contributor guide

Research direction
Remove `os name` from the `fields` tuple in the `Meta` class of `SurveyResultSerializer`. The explicit field declaration `os name = serializers.CharField(write only=True)` already defines the field; including it in `Meta.fields` causes a conflict because the model does not have that field.
Tech stack
pythondjango
Domain
backend
Issue type
Bug
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
2
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
Under 1 hour
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Active
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Clear
Prerequisites
PythonDjango
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
75