Skip to content

feat: remove selected_place from job results#11

Merged
1000hyehyang merged 2 commits into
devfrom
feat/10-remove-selected-place
May 2, 2026
Merged

feat: remove selected_place from job results#11
1000hyehyang merged 2 commits into
devfrom
feat/10-remove-selected-place

Conversation

@KyungminPark-steck
Copy link
Copy Markdown
Contributor

✨ 무엇을 바꿨나요?

selected_place 단일 필드를 제거하고, 장소 선택 결과를 selected_places 배열 기준으로 일원화했습니다.

🔗 관련 이슈

Closes #10

💡 왜 바꿨나요?

다중 장소 추출 구조에서는 selected_places가 실제 source of truth입니다.
selected_place는 기존 단일 장소 스키마 호환용으로 남겨둔 필드였지만, 팀 논의 결과 더 이상 유지할 필요가 없어 제거했습니다.

📝 주요 변경 사항

  • job_results.selected_place 컬럼 제거 migration 추가
  • API 응답 스키마에서 selected_place 제거
  • repository 저장/조회 로직에서 selected_place 제거
  • worker 처리 결과를 selected_places 기준으로 정리
  • 관련 테스트를 selected_places 기준으로 수정

👀 리뷰어가 보면 좋은 부분

selected_place만 제거되고, 단일/다중 장소 결과는 모두 selected_places에 배열로 유지됩니다.

  • 단일 장소: selected_places에 1개 저장
  • 다중 장소: selected_places에 여러 개 저장

🧪 테스트

방식 (해당하는 것만 체크)

  • 로컬 환경에서 확인
  • 운영 환경에서 확인
  • 단위 / 통합 테스트
  • 해당 없음

메모 (시나리오, 커맨드, 스크린샷 링크 등 — 선택)

  • python -m compileall app
  • .venv\Scripts\python.exe -m pytest -q
  • 결과: 57 passed, 1 skipped
  • 실제 HF/Kakao live 호출로 다중 장소 caption 1개, 단일 장소 caption 1개를 추가 확인했습니다.
    • 다중 장소: selected_places_count = 6
    • 단일 장소: selected_places_count = 1
    • 두 케이스 모두 selected_place key 없음

@1000hyehyang 1000hyehyang merged commit 0c6e4ce into dev May 2, 2026
1 check passed
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.

[feat] Remove selected_place

2 participants