Skip to content

feat: 원판 계산기 서버 API 연동 - #183

Merged
A6ul1213 merged 3 commits into
devfrom
feature/#182-plate-calculator-api-integration
Nov 4, 2026
Merged

A6ul1213 merged 3 commits into
devfrom
feature/#182-plate-calculator-api-integration

Conversation

@A6ul1213

@A6ul1213 A6ul1213 commented Nov 3, 2026 •

Copy link
Copy Markdown
Member

Summary

  • POST /api/v1/tools/plate-calculator를 원판 계산 화면의 ViewModel에 연결했습니다.
  • Debug에서는 로컬 Compose(http://localhost:8000)를 사용하고, Release에는 서버 연결을 추가하지 않았습니다.
  • 서버 응답의 원판 조합, 실제 총중량, 달성 불가 잔여 중량을 표시합니다.
  • 요청 실패 시 로컬 계산 결과를 유지하고 서버 오류 메시지와 같은 요청 재시도를 제공합니다.
  • ViewModel 성공·실패·재시도·입력 검증 테스트와 API 요청/응답 계약 테스트를 추가했습니다.

Related issue

Closes #182

Verification

  • Tuist project generation — 실행하지 않음
  • Build — 구현 당시 Debug 최종 빌드 및 Release 빌드 성공
  • Unit tests — 회귀 테스트 추가. 로컬 실행은 CoreSimulatorService 오류와 sandbox Swift 매크로 플러그인 실행 차단으로 완료하지 못함; CI 대기 중
  • SwiftLint — 실행하지 않음
  • Runtime or UI check — iPhone 17e Simulator에서 실패/재시도 상태 캡처 확인
  • 로컬 Compose 성공 응답 왕복은 Docker·Gradle 부재와 wrapper 다운로드 네트워크 차단으로 검증하지 못했습니다.

Review checklist

  • Scope is limited to the linked issue.
  • Tests cover changed behavior where appropriate.
  • No secrets, generated Xcode project files, or unrelated changes are included.

@dltldn1234 dltldn1234 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

요청 경로와 POST 메서드, 요청/응답 필드가 #182 및 API 계약 매핑과 일치하는 것을 확인했습니다. Debug 빌드에서만 localhost 서비스가 주입되고 Release에는 네트워크 구성이 추가되지 않았으며, 기존 네트워크 서비스 테스트도 요청 경로와 응답 디코딩을 확인합니다. CI의 Generate/Lint/Test 체크도 통과했습니다. 승인합니다. 참고로 이 PR에서 새 ViewModel 전용 회귀 테스트와 성공 응답의 실제 Compose 서버 왕복은 확인되지 않았습니다. 서버가 준비되면 성공 응답이 화면에 반영되는 흐름도 수동 검증해 주세요.

@A6ul1213
A6ul1213 marked this pull request as ready for review November 4, 2026 07:54
@A6ul1213
A6ul1213 merged commit 63bc762 into dev Nov 4, 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.

[Feature] 원판 계산기 서버 API 연동

2 participants