Skip to content

Conversation

@bbbang105
Copy link
Member

✅ PR 유형

어떤 변경 사항이 있었나요?

  • 새로운 기능 추가
  • 버그 수정
  • 코드에 영향을 주지 않는 변경사항(오타 수정, 탭 사이즈 변경, 변수명 변경)
  • 코드 리팩토링
  • 테스트코드 추가
  • 성능 개선 작업
  • 주석 추가 및 수정
  • 문서 수정
  • 빌드 부분 혹은 패키지 매니저 수정
  • 파일 혹은 폴더명 수정
  • 파일 혹은 폴더 삭제

🚀 작업 내용

이번 PR에서 작업한 내용을 구체적으로 설명해주세요. (이미지 첨부 가능)

주요 변경사항

  1. Platform enum 강제: MACOS, WINDOWS (대문자 필수, 잘못된 값은 400 에러)
  2. S3 Presigned URL: 30분 유효기간
  3. 업데이트 로그 기록: VERSION_CHECK, DOWNLOADED, UPDATE_SKIPPED 이벤트
  4. 강제 업데이트 지원: isMandatory 플래그
  5. Swagger 토큰 persist: 새로고침해도 Bearer 토큰 유지

신규 테이블

  1. app_release: 릴리즈 정보 (버전, 플랫폼, S3 경로, SHA-512 등)
  2. app_update_log: 업데이트 이벤트 로깅

신규 패키지

server.poptato.app/
├── api/           # Controller, Request DTO
├── application/   # Service, Response DTO
├── domain/        # Entity, Repository, Value Objects
└── status/        # Error Status

고민하다가 이후에 모바일 앱에도 해당 기능이 활용되기 때문에, 조금 더 범용적인 app을 택하였습니다.

테스트

  1. AppServiceTest: 12개 테스트 케이스
  2. AppControllerTest: 4개 테스트 케이스 (REST Docs 포함)

💬 기타 사항 or 추가 코멘트

남기고 싶은 말, 참고 블로그 등이 있다면 기록해주세요.

  • 테이블 DDL 등 자세한 내용은 노션 페이지 참고 부탁드립니다.

@bbbang105 bbbang105 self-assigned this Dec 29, 2025
@bbbang105 bbbang105 requested a review from woong2e as a code owner December 29, 2025 17:28
@bbbang105 bbbang105 added 🚀 feat 새로운 기능 추가 / 일부 코드 추가 / 일부 코드 수정 (리팩토링과 구분) / 디자인 요소 수정 😵‍💫 SANGHO 상호 Issue or PR labels Dec 29, 2025
@bbbang105 bbbang105 added the 🚨 fix 버그 수정 / 에러 해결 label Dec 29, 2025
Copy link
Member

@woong2e woong2e left a comment

Choose a reason for hiding this comment

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

고생하셨습니다 ! 👍

@bbbang105 bbbang105 added the 🔄 refactor 코드 리팩토링 label Jan 1, 2026
Copy link
Member

@woong2e woong2e left a comment

Choose a reason for hiding this comment

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

좋습니다 ! 👍

@sonarqubecloud
Copy link

sonarqubecloud bot commented Jan 9, 2026

Quality Gate Failed Quality Gate failed

Failed conditions
2 Security Hotspots
39.8% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud

@bbbang105 bbbang105 merged commit e46c2fa into develop Jan 12, 2026
4 of 5 checks passed
@bbbang105 bbbang105 deleted the feature/ILLDAN-199/desktop-update branch January 12, 2026 14:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🚀 feat 새로운 기능 추가 / 일부 코드 추가 / 일부 코드 수정 (리팩토링과 구분) / 디자인 요소 수정 🚨 fix 버그 수정 / 에러 해결 🔄 refactor 코드 리팩토링 😵‍💫 SANGHO 상호 Issue or PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants