improvement: 수정/삭제/진단 계약 확장 (#218)

seonkyu.kim commented

2026-02-25 04:37:36 +00:00

Owner

📋 작업 요약

플랫폼 자격증명 삭제 API 추가 (APNs/FCM 각각 별도 엔드포인트)
자격증명 진단 응답에 credentialStatus/statusReason 필드 추가
수정 API에 Status 필드 추가 (원자적 상태 변경 지원)
Swagger Description 업데이트

Closes #218

🛠️ 작업 내용 (Changes)

SPMS.Application/Interfaces/IServiceManagementService.cs — DeleteApns/FcmCredentialsAsync 인터페이스 추가
SPMS.Application/DTOs/Service/CredentialsResponseDto.cs — ApnsCredentialsInfoDto/FcmCredentialsInfoDto에 CredentialStatus, StatusReason 필드 추가
SPMS.Application/DTOs/Service/UpdateServiceRequestDto.cs — Status 필드 추가 (nullable int)
SPMS.Application/Services/ServiceManagementService.cs — 삭제 메서드 구현, credentials 진단 로직 추가, UpdateAsync에 Status 처리 추가
SPMS.API/Controllers/ServiceController.cs — APNs/FCM 삭제 엔드포인트 2개 추가, Swagger Description 갱신

📢 리뷰어 참고 사항 (To Reviewers)

APNs 삭제 시 p8/p12 관련 필드 8개 모두 null 처리
credentials 진단은 기존 BuildIosSummary/BuildAndroidSummary 로직을 재활용
Status 필드는 선택적 — 미제공 시 기존 동작과 동일

✅ 체크리스트 (Self Checklist)

빌드(Build)가 성공적으로 수행되었는가?
모든 단위 테스트(Unit Test)를 통과하였는가?
불필요한 로그나 주석을 제거하였는가?
컨벤션(Clean Architecture, Naming)을 준수하였는가?
기밀 정보(비밀번호, 키 등)가 하드코딩 되어있지 않은가?

📸 스크린샷 / 테스트 로그 (Screenshots/Logs)

빌드 성공: 경고 0개, 오류 0개

## 📋 작업 요약 - 플랫폼 자격증명 삭제 API 추가 (APNs/FCM 각각 별도 엔드포인트) - 자격증명 진단 응답에 `credentialStatus`/`statusReason` 필드 추가 - 수정 API에 `Status` 필드 추가 (원자적 상태 변경 지원) - Swagger Description 업데이트 ## 🔗 관련 이슈 (Related Issues) Closes #218 ## 🛠️ 작업 내용 (Changes) - [x] `SPMS.Application/Interfaces/IServiceManagementService.cs` — DeleteApns/FcmCredentialsAsync 인터페이스 추가 - [x] `SPMS.Application/DTOs/Service/CredentialsResponseDto.cs` — ApnsCredentialsInfoDto/FcmCredentialsInfoDto에 CredentialStatus, StatusReason 필드 추가 - [x] `SPMS.Application/DTOs/Service/UpdateServiceRequestDto.cs` — Status 필드 추가 (nullable int) - [x] `SPMS.Application/Services/ServiceManagementService.cs` — 삭제 메서드 구현, credentials 진단 로직 추가, UpdateAsync에 Status 처리 추가 - [x] `SPMS.API/Controllers/ServiceController.cs` — APNs/FCM 삭제 엔드포인트 2개 추가, Swagger Description 갱신 ## 📢 리뷰어 참고 사항 (To Reviewers) - APNs 삭제 시 p8/p12 관련 필드 8개 모두 null 처리 - credentials 진단은 기존 BuildIosSummary/BuildAndroidSummary 로직을 재활용 - Status 필드는 선택적 — 미제공 시 기존 동작과 동일 ## ✅ 체크리스트 (Self Checklist) - [x] 빌드(Build)가 성공적으로 수행되었는가? - [x] 모든 단위 테스트(Unit Test)를 통과하였는가? - [x] 불필요한 로그나 주석을 제거하였는가? - [x] 컨벤션(Clean Architecture, Naming)을 준수하였는가? - [x] 기밀 정보(비밀번호, 키 등)가 하드코딩 되어있지 않은가? ## 📸 스크린샷 / 테스트 로그 (Screenshots/Logs) 빌드 성공: 경고 0개, 오류 0개

seonkyu.kim added 1 commit 2026-02-25 04:37:37 +00:00

improvement: 수정/삭제/진단 계약 확장 (#218 ) 17caeb08e2

- 플랫폼 자격증명 삭제 API 추가 (APNs/FCM 각각)
- 자격증명 진단 응답에 credentialStatus/statusReason 추가
- 수정 API에 Status 필드 추가 (원자적 상태 변경)
- Swagger Description 업데이트

Closes #218