네트워크 통신 플로우 세팅 & 사용 가능 기프티콘 조회 api test #44
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
[작업 내용] - di, repository, service 세팅 - 사용 가능 기프티콘 조회 api test 및 데이터 정상 수신 확인 - 테스트용 버튼 클릭 시 api 통신 수행 [Todo] - 네트워크 통신 관련 코드 일원화 (data, network 모듈) - 데이터 맵핑 세부 수정
[작업 내용] - 개행 추가
|
@hanchang97 창희님 ktlint 에러 수정만 해주세요! |
ows3090
reviewed
Jan 17, 2024
| val isLastPage: Boolean = false, | ||
| val pageNumber: Int = 0, | ||
| ) { | ||
| data class AvailableGifticonInfo( |
There was a problem hiding this comment.
@hanchang97 클래스 안에 클래스가 있는 구조의 장점이 있나요?
There was a problem hiding this comment.
중첩된 data class 사용을 통해 관련된 데이터 모델을 논리적으로 그룹화하려는 의도였습니다. 중첩된 data class는 외부 클래스와 강한 연관성을 가지기 때문에, 해당 클래스가 어떤 목적을 가지는지 명확하게 표현할 수 있을거라 생각했습니다..!
| ) | ||
| } | ||
|
|
||
| data class Pageable( |
There was a problem hiding this comment.
@hanchang97
Pageable , SortX 클래스는 필요하지 않을 것 같은데 왜 모델링 하셨을까요?
There was a problem hiding this comment.
응답 정상적으로 오는지 확인하는 과정에서 일단 만들어뒀는데 사양적으로 필요하지 않을 것 같아서 제거하도록 하겠습니다!
[작업 내용] - 불필요 개행 제거
[작업 내용] - 불필요 개행 제거 - 개행 추가 - 변수 타입 선언 시 ':' 전의 공백 제거 - 주석 표시 후 공백 추가
[작업 내용] - 불필요 파일 제거 - 라인 최대 길이 수정
[작업 내용] - 들여쓰기 수정 16 -> 12
|
@hanchang97 페이지쪽 데이터만 지우면 머지하겠습니다! |
[작업 내용] - 불필요 데이터 제거
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
[작업 내용]
[ 데이터 수신 로그 ]
[Todo]
구체적인 작업 내용
화면 ( option - UI 관련 작업시에만 )
버그 수정시 정상 작동 case ( option - 버그 fix 시에 )
기타 사항
Close Issue
Close #