-
Notifications
You must be signed in to change notification settings - Fork 117
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.
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
오픈 마켓 [STEP 1] 아리,제리 #82
Merged
Merged
Changes from 1 commit
Commits
Show all changes
43 commits
Select commit
Hold shift + click to select a range
ca846b3
style: SwfitLint 라이브러리 추가
leeari95 d04a121
feat: Products 타입 추가 #1
llghdud921 2b14a79
feat: Product 타입 추가 #1
leeari95 f57c0e3
chore: products 파일 Assests에 추가
llghdud921 82fafce
feat: 파싱해주는 타입 Parser을 추가 #1
leeari95 a8e9388
test: ParsetTest 추가 #2
llghdud921 59f34d1
test: product를 파싱하는 테스트 추가 #2
leeari95 c4406dd
feat: network class 타입 및 NetworkError 추가 #3
llghdud921 4c22c0a
feat: URL과 HTTPMethod를 담고있는 타입 추가 #3
leeari95 2af414a
feat: NetworkManager 추가 #3
llghdud921 4446fd0
feat: 상품 리스트 조회시 필요한 request를 반환하는 메소드 추가 #3
leeari95 826aaf3
feat: 상품 상세 조회를 위한 request메서드 추가 #3
llghdud921 2679eef
feat: 상품의 Secret을 조회할 때 필요한 Request 반환하는 메소드 추가 #3
leeari95 c67be7b
feat: 상품 삭제를 위한 request 메소드 추가 #3
llghdud921 0cefe03
feat: 상품 수정을 위한 request 메소드 추가 #3
leeari95 b088b10
feat: 상품 등록을 위한 request 추가 #3
llghdud921 be0cb43
refactor: NetworkManager 내부 리팩토링 #4
leeari95 27fa33f
refactor: NetworkError 내부 리팩토링 및 ProductSecret 타입명 변경 #4
llghdud921 e73b5fc
refactor: MultipartForm 프로토콜을 Protocol 그룹으로 이동 #4
leeari95 a54a4ce
refactor: Products 내 CodingKeys 수정 #4
llghdud921 3294a3d
test: Network의 excute 메소드를 테스트하는 코드 추가 #5
leeari95 7872ab5
test: NetworkManager의 fetch를 테스트하는 코드 추가 #5
llghdud921 cacce89
test: URLProtocol을 재정의하여 테스트 코드를 전반적으로 수정 #5
leeari95 aa5b4d2
refactor: NetworkManager내 multipart image 경로 수정 #4
llghdud921 fb39366
test: 상품리스트 조회에 맞는 Request를 반환하는 테스트코드 추가 #5
leeari95 b0eb1e0
test: 상품 상세 조회를 반환하는 request를 확인하는 테스트코드 추가 #5
llghdud921 e4cdd8a
test: 상품 삭제 Secret 조회에 맞는 Request를 반환하는 테스트코드 추가 #5
leeari95 7f373fc
test: 상품 삭제를 반환하는 request를 확인하는 테스트코드 추가 #5
llghdud921 a245770
test: 상품 수정에 맞는 Request를 반환하는 테스트코드 추가 #5
leeari95 42a798d
test: 상품 등록을 반환하는 request를 확인하는 테스트코드 추가 #5
llghdud921 1bde0bd
refactor: multipartFormRequest 메소드 내부에 request의 헤더를 추가하는 부분 수정 #4
leeari95 cfd7f09
refactor: MockURLProtocol내 주석 제거 #4
llghdud921 79c207a
refactor: NetworkConstant를 Address와 HTTPMethod로 분리 #4
leeari95 69352ee
docs: Step1 README 작성
llghdud921 164da38
docs: README.md 파일 수정
leeari95 afa5c2a
refactor: 테스트 코드에 중복되는 부분 리팩토링 #6
leeari95 dd21787
refactor: test code내 주석 및 줄바꿈 수정 #6
llghdud921 7bd46d8
refactor: Image의 프로퍼티 네이밍을 명확하게 수정 #6
leeari95 513ae69
refator: Content-type,MultipartForm String값 상수로 정의 #6
llghdud921 e3b81a7
refactor: 에러의 케이스중 네이밍을 명확하게 개선 #6
leeari95 05fddcb
refactor: Parser, Parsable에 JSON을 추가한 네이밍으로 개선 #6
llghdud921 e44134c
refactor: Network 프로퍼티에 접근제어 부여하여 개선 #6
leeari95 d553f99
refactor: Address를 APIAdress로 네이밍 개선 #6
llghdud921 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
refactor: MockURLProtocol내 주석 제거 #4
- Loading branch information
commit cfd7f09bfe65a7bc51f63ff73607a8c5c4c9be5d
There are no files selected for viewing
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
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
오버라이드해주고 아무 구현이 없는것 같아요!
추후 추가되는건가요?
그게 아니라면 현재로써는 불필요한 코드로 보입니다🤔
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
아까 말씀드린대로 애플에서 구현없이 두고있어서 해당 코드 유지하도록 하겠습니다!☺️