Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[Network] Request(Response) Model 일부 리팩토링 #256

Merged
merged 34 commits into from
Apr 6, 2023
Merged

[Network] Request(Response) Model 일부 리팩토링 #256

merged 34 commits into from
Apr 6, 2023

Conversation

WhiteHyun
Copy link
Member

@WhiteHyun WhiteHyun commented Apr 5, 2023 •
edited
Loading

📌 PR 요약

🌱 작업한 내용

  • 대부분의 요청(응답)모델에 주석을 추가하였고, 불필요한 코드도 삭제했습니다.

아래부터는 담당 파트 별 알고있어야할 수정 사항 내역입니다.

  • 건준

    • b1c6cc7
    • 6dde5db
    • b301765
    • 3262d28
    • MainCategory와 Category통합을 하려했는데 얽힌 코드가 있어 제가 따로 수정해두진 않았습니다. 이 PR 이후에 리팩토링 진행해주시면 좋겠어요.

  • 수빈

🌱 PR 포인트

  • FeedsPaginatedDataResponse를 PaginatedDataResponse로 이름을 변경하였습니다. 이제 페이징 처리할 때 PaginatedDataResponse를 사용하여 작업해주세요. 아래는 PaginatedDataResponse로 수정해야할 모델 리스트입니다.
    1. CategoryMeetingResponse
    2. SearchRecruitmentResponse
    3. BookmarkAllResponse
  • 공통으로 사용되는 모델은 General Models라는 폴더를 따로 두어 정리해두었습니다. 해당 폴더를 참고해주세요.

📸 스크린샷

아래 표시된 곳 빼고 작업했어요
image
  • 코드 보는 게 너무 힘들어서... 박스친 곳은 나중에 작업할게요.. 힘들어요.

📮 관련 이슈

@WhiteHyun
[CHORE] Foldering
122b4b4 
알파벳 순으로 정렬
@WhiteHyun
[CHORE] InquireInterestResponse's property
b1c6cc7 
categoryID가 [Int] 타입이기에 s를 붙여 단일 요소처럼 보이지 않도록 처리함
@WhiteHyun
[ADD] Comments
29b98ec 
주석을 단 모델 리스트
- RegisterInterestRequest
- InquireInterestResponse
- MyInfoResponse
@WhiteHyun
[RENAME] Rename SignInType to SocialType
0fc88fc
@WhiteHyun
[CHORE] Change the property type of MyInfoResponse
896b55e
@WhiteHyun
[ADD] Comments
db9c66f 
- MyInfoResponse
- SignInRequest
- SignUpRequest
- SignInResponse
- TokenResponse
@WhiteHyun
[ADD] Comments
308a999 
- AllCategoryListResponse
- MainCategoryListResponse
- SubCategoryListResponse
@WhiteHyun
[DEL] Remove unnecessary code
84f6a4c 
- 불필요한 CodingKeys
- isSelected는 API에서 보내는 값이 아니므로 디코딩할 필요가 전혀 없습니다.
@WhiteHyun
[ADD] Comments
5a9115f 
- BoardsRequest
- CommentsRequest
- BoardsResponse
- FeedsClipboardResponse
@WhiteHyun
[ADD] Comments
b4020cd 
- BoardsRequest
- CommentsRequest
- BoardsResponse
- FeedsClipboardResponse
- FeedsContent
@WhiteHyun
[CHORE] Change fileURL property of Files
0d4d56a
@WhiteHyun
[CHORE] OnOff to MeetType
9e32769
@WhiteHyun
[CHORE] Change subCategoryID to subCategoryIDs
6dde5db
@WhiteHyun
[CHORE] Add optional type in CategoryMeetingRequest
b301765 
days와 subCategoryIDs는 필수 값이 아니기에 옵셔널 타입을 추가하였습니다.
@WhiteHyun
[CHORE] Comments in EditMeetingInfoRequest
5bd3e17
@WhiteHyun
[ADD] Add Comments
60efb0c 
- CreateMeetingResponse
- MyMeetingResponse
@WhiteHyun
[CHORE] cursorId to cursorID in MyPlubbingParameter
2ebb2c6
@WhiteHyun
[ADD] Comments
41469cf 
- PlubbingStatusType
- MyPlubbingResponse
- MyPagePlubbing
@WhiteHyun
[REFACTOR] Refactor status field in MyPlubbingParameter to use Plubbi…
0ae5223 
…ngStatusType enum

원래 MyPlubbingResponse의 plubbingStatus 필드는 PlubbingStatusType.RawValue 타입이었고,
MyPlubbingParameter의 status 필드는 String 타입이었습니다.
그러나 status에 전달되는 값은 PlubbingStatusType 열거형의 값과 동일하여 일관성을 위해 열거형으로 통합했습니다.
@WhiteHyun
[DEL] Remove unnecessary CodingKeys
79b534c
@WhiteHyun
[ADD] Add Comments in SearchParameter
a2f0e96
@WhiteHyun
[CHORE] Deprecate page field in SearchParameter
3262d28 
page 프로퍼티는 이제 사용되지 않아 deprecated 처리했습니다.
페이징 처리를 따로 구현해주시길 바랍니다: @dlrjswns
@WhiteHyun
[DEL] Remove unnecessary CodingKeys in EditMeetingPostRequest
35b924d
@WhiteHyun
[ADD] Add Comments
3689274
@WhiteHyun
[ADD] Add Comments in EditMeetingQuestionRequest
776af03
@WhiteHyun
[DEL] Remove unnecessary CodingKeys in EditMeetingQuestionRequest
a441d0b
@WhiteHyun
[ADD] Add Comments
0cc8f92 
- ApplyForRecruitmentRequest
- ApplyAnswer
@WhiteHyun
[ADD] Add comments
70d3357 
- DetailRecruitmentResponse
- AccountInfo
- RecruitmentQuestionResponse
- Question
- RequestBookmarkResponse
- ApplyForRecruitmentResponse
- InquireApplicantResponse
- Application
- Answer
@WhiteHyun
[DEL] Remove unnecessary CodingKeys in Answer
c2b5d65
@WhiteHyun
[ADD] Add comments
53ae74b 
- MyApplicationResponse
- PlubbingInfo
@WhiteHyun
[RENAME] FeedsPaginatedDataResponse to PaginatedDataResponse
171d3f8
@WhiteHyun
[DEL] Unused models
7c0f90c 
- Pageable, Sort, Plubbing 제거
@WhiteHyun
[ADD] Add comments in Content
a12c62c
@WhiteHyun
[MOVE] Create GeneralModels and move files using general models
7ed78c6 
성별, 소셜 로그인 타입, 페이징 처리 모델과 응답 모델을 GeneralModels 폴더 내부에 두어
공통으로 처리되는 모델을 한 군데 모아두었습니다.
@WhiteHyun WhiteHyun self-assigned this Apr 5, 2023
@WhiteHyun WhiteHyun added the D+1 PR올린 다음날 까지는 리뷰 부탁드립니다 label Apr 5, 2023
dlrjswns
dlrjswns approved these changes Apr 5, 2023
View reviewed changes
Copy link
Contributor

@dlrjswns dlrjswns left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

와우 말씀하신것처럼 변경된게 엄청 나네요...ㅋㅋㅋㅋㅋㅋㅋ 저도 바통터치하겠습니다 👍

@WhiteHyun WhiteHyun merged commit 75b2d6a into develop Apr 6, 2023
@WhiteHyun WhiteHyun deleted the feat/255-Network/Refactoring branch April 6, 2023 01:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dlrjswns dlrjswns dlrjswns approved these changes

@soobin-k soobin-k Awaiting requested review from soobin-k

Assignees

@WhiteHyun WhiteHyun

Labels
D+1 PR올린 다음날 까지는 리뷰 부탁드립니다 ♻️Refactoring
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@WhiteHyun @dlrjswns