Skip to content

fix: API 테스트 서버 scheme 추가#5

Merged
ysw789 merged 1 commit into
mainfrom
fix/alt-3-1
Apr 20, 2025
Merged

fix: API 테스트 서버 scheme 추가#5
ysw789 merged 1 commit into
mainfrom
fix/alt-3-1

Conversation

@ysw789
Copy link
Copy Markdown
Contributor

@ysw789 ysw789 commented Apr 19, 2025

ID

  • ALT-3

변경 내용

  • swagger-ui 서버 scheme 추가

구현 사항

  • 실제 서버 API 엔드포인트를 자동으로 참조하도록 하여 swagger-ui 상에서 API 테스트를 진행할 수 있도록 구현

- swagger-ui로 배포된 API 테스트 가능하도록 수정
@ysw789 ysw789 added the FIX label Apr 19, 2025
@ysw789 ysw789 self-assigned this Apr 19, 2025
@ysw789 ysw789 merged commit 6cb0239 into main Apr 20, 2025
ysw789 added a commit that referenced this pull request May 10, 2026
- UpdateWorkspaceWorkerColorCommand 도메인 Command 클래스 신규 생성
- 인바운드 포트(ManagerUpdateWorkspaceWorkerColorCodeUseCase)에서 어댑터 DTO 의존 제거
- UseCase 구현체에서 Command 사용으로 변경
- 컨트롤러에서 DTO → Command 변환 후 호출 (어댑터 검증 책임 유지)

헥사고날 아키텍처 경계 준수 (CodeRabbit 리뷰 #2, #5 반영)
ysw789 added a commit that referenced this pull request May 10, 2026
- UpdateWorkspaceWorkerColorCommand 도메인 Command 클래스 신규 생성
- 인바운드 포트(ManagerUpdateWorkspaceWorkerColorCodeUseCase)에서 어댑터 DTO 의존 제거
- UseCase 구현체에서 Command 사용으로 변경
- 컨트롤러에서 DTO → Command 변환 후 호출 (어댑터 검증 책임 유지)

헥사고날 아키텍처 경계 준수 (CodeRabbit 리뷰 #2, #5 반영)
ysw789 added a commit that referenced this pull request May 10, 2026
* feat: 근무자 색상 코드 엔티티 필드 및 UseCase 추가

* feat: 근무자 색상 코드 변경 DTO 및 응답 필드 추가

* feat: 근무자 색상 코드 변경 API 엔드포인트 추가

* refactor: Command 패턴으로 UseCase 아키텍처 개선

- UpdateWorkspaceWorkerColorCommand 도메인 Command 클래스 신규 생성
- 인바운드 포트(ManagerUpdateWorkspaceWorkerColorCodeUseCase)에서 어댑터 DTO 의존 제거
- UseCase 구현체에서 Command 사용으로 변경
- 컨트롤러에서 DTO → Command 변환 후 호출 (어댑터 검증 책임 유지)

헥사고날 아키텍처 경계 준수 (CodeRabbit 리뷰 #2, #5 반영)

* docs: ControllerSpec ErrorCode 예시 정합성 수정

- 409 응답의 불명시 ErrorCode B026 → B020(CONFLICT)으로 수정
- 실제 UseCase가 throw하는 ErrorCode.CONFLICT와 일치하도록 정정
- 404 응답 예시(B008 업장, B019 근무자)는 정의된 코드로 유지

Swagger 문서 정합성 개선 (CodeRabbit 리뷰 #1 + 추가 발견 반영)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant