-
Notifications
You must be signed in to change notification settings - Fork 3
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
[iOS] 이슈 뷰 구현 #54
Merged
Merged
[iOS] 이슈 뷰 구현 #54
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
diffableDatasource 구현 필요
이슈 모델 수정 - UserID와 User에 대해 옵셔널 추가 코멘트 구조체에 셀 구별을 위한 Hashable 프로토콜 구현
- 사용되지 않는 코드 삭제
- LoginVC 생성 메소드를 lazy var로 변경
- header용 view 대신 navigationBar를 적용 - large title 사용
- 세부사항 bottomSheetVC 추가
- manager 관련 파일 이동
- Issue.swift에 있는 모델 객체 개별 파일들로 분리 - IssueElement 삭제
- NetworkManager 추가에 대한 변경 - GithubLoginManager 이름 변경에 따른 변경
- 모델 파일 추가에 의한 project.pbxproj 변경
뷰모델 구조 변경 날짜 형식에 맞게 표시되도록 표시 수정
API에 따른 모델 옵셔널 처리 이슈 상세보기 더미 삭제
[iOS] 이슈 상세보기 화면 구현
…_cell' into feature/ios_issue
네비게이션 컨트롤러 추가에 따라서 변경
- 모델에 Hashable 적용 - 필요 없는 ViewModel 관련 코드 삭제 - 변경된 내용에 맞게 이슈 데이터 다운로드 로직 변경 - 그 외 변수 이름 등 변경
…acker-01 into feature/ios_issue
- 최대한 일관된 스타일을 갖도록 조정 - 나머지는 상의 후 재조정
- itemSetHandler -> issueChangeHandler - CommentCollectionViewCell ->IssueCommentCell - 강제 언래핑 변경
- 반복되는 데이터 다운로드 제거 - 그 외 관련 사항 수정
- 프로퍼티 감시자 제거 후, load시 applySnapshot하도록 변경
- 기본적인 뷰 표시상태 구성
이슈, 레이블, 마일스톤, 댓글에 대한 네트워킹 함수 구현
- 이슈 셀의 레이블 표시 오류 수정
label 변수명을 맞추거나 파라미터를 재가공할 필요 있음
…acker-01 into feature/ios_issue
이슈 관련 서버 통신을 위한 파라미터 구조체 구현
- 테스트 코드 삭제 - storyboard entrypoint 수정
- nav -> navigationViewController
- 일반 view에서 reusable view를 이용해 헤더 사용 - 기능이 없으므로, 런타임땐 표시되지않음
이슈 정보 모달 뷰에서 사용할 마일스톤 진행바 구현
- 이슈를 닫고, 이슈 메인으로 이동
…/IssueTracker-01 into feature/ios_issue # Conflicts: # ios/IssueTracker/IssueTracker/Utilitys/NetworkManager.swift
This was
linked to
issues
Nov 12, 2020
Closed
Closed
Closed
와 엄청 많이 작업하셨네요!! 수고하셨습니다~ 🤗 |
- 필터 뷰를 통해 이슈 필터링 가능 - open, close 필터링 가능
7 tasks
- 키워드를 통한 이슈 탐색 기능 추가
- 선택한 셀들의 이슈 상태를 close로 변환
Closed
3 tasks
- flexible space를 이용해 버튼 위치 오른쪽으로 이동
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.
이슈 다중선택 편집 화면
이슈 세부목록 뷰 구현
이슈 세부 목록 바텀 시트 구현
이슈 필터 선택 모달 뷰 구현
필터링 기능 구현
이슈 생성/수정 뷰 구현
이슈 세부목록 셀 구현
기타