Skip to content

Conversation

@daheeParkk
Copy link
Contributor

@daheeParkk daheeParkk commented Mar 3, 2024

▶ Request

Content

as-is

삭제 해제하고 싶은 상점을 삭제되지 않은 상점 중에서 찾고 있었습니다.

to-be

상점 조회할 때 삭제된 상점도 조회되도록 수정하였습니다.

✅ Check List

  • 의도치 않은 변경이 일어나지 않았는지.
    • 실수로 라이브러리(pom.xml) 변경이 일어나지 않았는지
    • 병합시 컴파일 & 런타임 에러가 발생하지 않는지
    • 기존 클라이언트와의 호환성 고려가 잘 이루어졌는지
  • 작성한 코드가 프로젝트에 반영됨을 명심하였는지
    • 타인도 알아보고 변경할 수 있는 코드를 작성하였는지
    • 코드 & 커밋 컨벤션을 준수하였는지
    • (필요한) 문서화가 진행되었는지
  • (기능 추가의 경우) 클라이언트의 입장에 대한 충분한 고려가 이루어졌는지
    • 클라이언트 측과 협의가 된 내용인 경우
    • 클라이언트의 요구사항을 잘 반영하는지
    • API 문서에 논리적인 오류 & 가시성이 떨어지는 내용이 없는지

📸 API Document ScreenShot

image

🧪 Test

  • 삭제된 상점을 삭제 해제하는 과정을 테스트하였습니다.

@daheeParkk daheeParkk requested a review from Choi-JJunho March 3, 2024 13:19
@daheeParkk daheeParkk self-assigned this Mar 3, 2024
@Choi-JJunho Choi-JJunho merged commit 8b6f6cf into develop Mar 3, 2024
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.

3 participants