fix: Public API에서 X-Service-Code 의존성 제거 (#90) #91

Merged
seonkyu.kim merged 1 commits from fix/#90-remove-service-code-public into develop 2026-02-10 05:29:50 +00:00
Owner

📋 작업 요약

  • Public API(공지사항, 배너, FAQ, 약관/개인정보, 앱 버전, 앱 설정)에서 X-Service-Code 헤더 의존성 완전 제거
  • Public API는 SPMS 플랫폼 공용 API로 서비스별 분리가 불필요

Closes #90

🛠️ 작업 내용 (Changes)

  • NoticeController.cs — X-Service-Code 헤더 체크 및 serviceCode 파라미터 제거
  • BannerController.cs — X-Service-Code 헤더 체크 및 serviceCode 파라미터 제거
  • FaqController.cs — X-Service-Code 헤더 체크 및 serviceCode 파라미터 제거
  • TermsController.cs — X-Service-Code 헤더 체크 및 serviceCode 파라미터 제거
  • AppVersionController.cs — X-Service-Code 헤더 체크 및 serviceCode 파라미터 제거
  • AppConfigController.cs — X-Service-Code 헤더 체크 및 serviceCode 파라미터 제거
  • INoticeService.cs / NoticeService.cs — serviceCode 파라미터 및 IServiceRepository 의존성 제거
  • IBannerService.cs / BannerService.cs — serviceCode 파라미터 및 IServiceRepository 의존성 제거
  • IFaqService.cs / FaqService.cs — serviceCode 파라미터 및 IServiceRepository 의존성 제거
  • IAppConfigService.cs / AppConfigService.cs — serviceCode 파라미터 및 IServiceRepository 의존성 제거
  • INoticeRepository.cs / NoticeRepository.cs — serviceId 파라미터 및 필터 제거
  • IBannerRepository.cs / BannerRepository.cs — serviceId 파라미터 및 필터 제거
  • IFaqRepository.cs / FaqRepository.cs — serviceId 파라미터 및 필터 제거
  • IAppConfigRepository.cs / AppConfigRepository.cs — serviceId 파라미터 제거, GetAllByServiceAsync → 기본 GetAllAsync 사용

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

  • 4계층(Controller → Service Interface → Service → Repository Interface → Repository) 전체에서 serviceId/serviceCode 의존성을 일괄 제거
  • AppConfigRepository의 GetAllAsync()는 기본 IRepository의 GetAllAsync()를 그대로 활용 (중복 선언 제거)
  • 22개 파일 변경, 빌드 경고/오류 0개

체크리스트 (Self Checklist)

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

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

빌드했습니다.
    경고 0개
    오류 0개
## 📋 작업 요약 - Public API(공지사항, 배너, FAQ, 약관/개인정보, 앱 버전, 앱 설정)에서 X-Service-Code 헤더 의존성 완전 제거 - Public API는 SPMS 플랫폼 공용 API로 서비스별 분리가 불필요 ## 🔗 관련 이슈 (Related Issues) Closes #90 ## 🛠️ 작업 내용 (Changes) - [x] `NoticeController.cs` — X-Service-Code 헤더 체크 및 serviceCode 파라미터 제거 - [x] `BannerController.cs` — X-Service-Code 헤더 체크 및 serviceCode 파라미터 제거 - [x] `FaqController.cs` — X-Service-Code 헤더 체크 및 serviceCode 파라미터 제거 - [x] `TermsController.cs` — X-Service-Code 헤더 체크 및 serviceCode 파라미터 제거 - [x] `AppVersionController.cs` — X-Service-Code 헤더 체크 및 serviceCode 파라미터 제거 - [x] `AppConfigController.cs` — X-Service-Code 헤더 체크 및 serviceCode 파라미터 제거 - [x] `INoticeService.cs` / `NoticeService.cs` — serviceCode 파라미터 및 IServiceRepository 의존성 제거 - [x] `IBannerService.cs` / `BannerService.cs` — serviceCode 파라미터 및 IServiceRepository 의존성 제거 - [x] `IFaqService.cs` / `FaqService.cs` — serviceCode 파라미터 및 IServiceRepository 의존성 제거 - [x] `IAppConfigService.cs` / `AppConfigService.cs` — serviceCode 파라미터 및 IServiceRepository 의존성 제거 - [x] `INoticeRepository.cs` / `NoticeRepository.cs` — serviceId 파라미터 및 필터 제거 - [x] `IBannerRepository.cs` / `BannerRepository.cs` — serviceId 파라미터 및 필터 제거 - [x] `IFaqRepository.cs` / `FaqRepository.cs` — serviceId 파라미터 및 필터 제거 - [x] `IAppConfigRepository.cs` / `AppConfigRepository.cs` — serviceId 파라미터 제거, GetAllByServiceAsync → 기본 GetAllAsync 사용 ## 📢 리뷰어 참고 사항 (To Reviewers) - 4계층(Controller → Service Interface → Service → Repository Interface → Repository) 전체에서 serviceId/serviceCode 의존성을 일괄 제거 - AppConfigRepository의 GetAllAsync()는 기본 IRepository<T>의 GetAllAsync()를 그대로 활용 (중복 선언 제거) - 22개 파일 변경, 빌드 경고/오류 0개 ## ✅ 체크리스트 (Self Checklist) - [x] 빌드(Build)가 성공적으로 수행되었는가? - [x] 모든 단위 테스트(Unit Test)를 통과하였는가? - [x] 불필요한 로그나 주석을 제거하였는가? - [x] 컨벤션(Clean Architecture, Naming)을 준수하였는가? - [x] 기밀 정보(비밀번호, 키 등)가 하드코딩 되어있지 않은가? ## 📸 스크린샷 / 테스트 로그 (Screenshots/Logs) ``` 빌드했습니다. 경고 0개 오류 0개 ```
seonkyu.kim added 1 commit 2026-02-10 05:27:50 +00:00
seonkyu.kim added the
Priority
High
Status
In Progress
Type
Bug
labels 2026-02-10 05:28:11 +00:00
seonkyu.kim self-assigned this 2026-02-10 05:28:12 +00:00
seonkyu.kim added this to the Phase 2-2: Public & 디바이스 & 파일 API milestone 2026-02-10 05:28:15 +00:00
seonkyu.kim requested review from Owners 2026-02-10 05:28:31 +00:00
seonkyu.kim merged commit 31df012976 into develop 2026-02-10 05:29:50 +00:00
seonkyu.kim deleted branch fix/#90-remove-service-code-public 2026-02-10 05:30:16 +00:00
seonkyu.kim added
Status
Done
and removed
Status
In Progress
labels 2026-02-10 05:30:33 +00:00
Sign in to join this conversation.
No description provided.