Skip to content

feat: 구매 고민 입력 폼 구현 - #12

Open
jiyeoniii wants to merge 1 commit into
developfrom
feat/products-consideration-form
Open

feat: 구매 고민 입력 폼 구현#12
jiyeoniii wants to merge 1 commit into
developfrom
feat/products-consideration-form

Conversation

@jiyeoniii

Copy link
Copy Markdown
Collaborator

📌 작업 내용

docs/API.md §5.1 구현
상품명, 가격을 사용자가 직접 입력

  • ConsiderationForm 추가 — 가격 1원 이상, 카테고리 최대 3개, purpose=ETC일 때 purpose_detail 필수, 비활성 카테고리 선택 불가
  • ConsiderationCreateView 추가 (LoginRequiredMixin + CreateView)
  • 페이지/JSON API URL 분리 (§2.2) — config/api_urls.py 신설, 기존 /api/accounts/... 경로와 reverse 이름은 그대로 유지
  • MAX_CATEGORY_SELECTION, LOGIN_URL 설정 추가
  • price_input.js — 입력 중 천 단위 구분 기호 표시와 한글 금액 되읽기, 제출 시 정수로 변환
  • 폼 검증 규칙 테스트 10개 추가

성공 시 리다이렉트 대상인 대안 생성 페이지는 아직 없어 URL 이름 대신 경로를 직접 사용

✅ 체크리스트

  • 로컬에서 정상적으로 동작하는지 확인했습니다.
  • 프론트와 백엔드 간 데이터 형식을 확인했습니다. (가격은 정수로만 전송)
  • URL 및 템플릿 변수명을 확인했습니다.
  • 마이그레이션 파일이 필요한 경우 포함했습니다. (모델 변경 없어 불필요)

💬 리뷰 요청 사항

  • 성공 리다이렉트가 아직 404입니다. 문서대로 /alternatives/considerations/<id>/로 보내는데 alternatives 페이지가 없어서, URL 이름 대신 경로를 직접 썼습니다. 해당 작업 때 reverse()로 바꿔주세요!
  • 확인하려면 /admin/으로 로그인 후 /products/considerations/new/로 가야 합니다.
  • static/css/consideration_form.css는 최소 스타일만 넣은 신규 파일입니다. FE 디자인에 맞춰 자유롭게 교체해주세요!!
  • products/api_urls.py는 §5.3(비교 기준 변경 PATCH)용 빈 스텁입니다.

📷 스크린샷

스크린샷 2026-08-02 004116

docs/API.md §5.1 구현입니다. 상품명·가격을 사용자가 직접 입력합니다.

- ConsiderationForm 추가 — 가격 1원 이상, 카테고리 최대 3개,
  purpose=ETC일 때 purpose_detail 필수, 비활성 카테고리 선택 불가
- ConsiderationCreateView 추가 (LoginRequiredMixin + CreateView)
- 페이지/JSON API URL 분리 (§2.2) — config/api_urls.py 신설,
  기존 /api/accounts/... 경로와 reverse 이름은 그대로 유지
- MAX_CATEGORY_SELECTION, LOGIN_URL 설정 추가
- price_input.js — 입력 중 천 단위 구분 기호 표시와 한글 금액
  되읽기, 제출 시 정수로 변환
- 폼 검증 규칙 테스트 10개 추가

성공 시 리다이렉트 대상인 대안 생성 페이지는 아직 없어 URL 이름 대신
경로를 직접 사용합니다.
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.

1 participant