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.

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

main2 #17

Open
wants to merge 125 commits into
base: main
Choose a base branch
from
Open

main2 #17

wants to merge 125 commits into from

Conversation

ChaminLee
Copy link
Owner

No description provided.

ChaminLee and others added 30 commits January 3, 2022 16:49
- SwiftLint disabled_rules에 trailing_whitespace 추가
- 모델 프로퍼티 오타 수정
- SwiftLint yml 파일 수정
- HttpMethod 타입 구현
- URLRequest extension하여 get,post,delete,patch 구현
- Generics와 Result 타입을 활용하여 Decodable을 채택한 타입이 들어올 수 있도록 구현
- Mock Data 변경
- 테스트 코드 변경
- 모델 프로퍼티 타입 변경
- ProductDetail내 옵셔널 프로퍼티로 추가
- APIService session 프로퍼티 생성 후 URLSessionProtocol 채택
- APIService 내에 생성자 주입방식으로 URLSession 주입
- MockURLSessionDataTask 이니셜라이저 구현
- Currency 타입 분리
- URLManager 타입 생성
- generateBoundary 메소드 구현
- convertDataToMultiPartForm 메소드 구현
- Data extension하여 appendString 메소드 구현
- ImageData 타입 구현
- createBody에 이미지 파일도 추가하도록 변경
- Request 폴더 생성
- 공통된 dataTask 메서드 분리
- APIError 구현
- URLCreator 케이스 추가
- MockURLSession에 APIError 추가
yim2627 and others added 30 commits January 24, 2022 23:19
- 상품 페이지 넘버 구현
- 메인 화면 리스트 레이아웃 조정
- 이미지 슬라이더 stackview에서 분리
- 등록뷰의 이미지 스택뷰 속성 변경
- NotificationKey 타입 추가
- UserInformation 타입 추가
- 본인의 상품이 아닐 때 수정/삭제 불가능하도록 기능 구현
- 상품 secret 조회 기능 로직 수정(header추가, body 타입 변경)
- 상품 삭제 결과에 따른 alert 구현
- ProductSecret 타입 구현
- 상품 등록/수정 ViewController는 ProductManageViewController 상속
- AlertMessage 추가
- productRegisterManager내 판별 로직 추가
- productManageable 제거
- 화면별로 폴더 분리
- checkValidInput -> checkValidReceivedProductDetail 네이밍 변경
- LoadingCollectionViewReusableView 구현
- 다음 페이지가 존재할 경우 새로운 데이터 추가
- CollectionView List Layout 구현 방식 변경
- collectionView 스크롤 버벅임 해결
- prefetch에서 다음 페이지의 데이터를 불러오도록 구현
- cell 높이 고정적으로 변경
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.

2 participants