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

✅ [CHORE] BaseQuestionTVC관련 중복 파일을 제거하고 TVC를 통일합니다. #544

Merged
merged 74 commits into from
Oct 19, 2022
Merged

✅ [CHORE] BaseQuestionTVC관련 중복 파일을 제거하고 TVC를 통일합니다. #544

merged 74 commits into from
Oct 19, 2022

Conversation

hwangJi-dev
Copy link
Member

@hwangJi-dev hwangJi-dev commented Oct 16, 2022 •
edited
Loading

🍎 관련 이슈

closed #542

🍎 변경 사항 및 이유

  • BaseQuestionTVC의 UI가 중복되어 override되고 있고, 더이상 필요하지 않은 TVC들을 사용하고 있는 곳들이 있어서 해당 코드들을 통일해주었습니다.

🍎 PR Point

  • 필요없는 중복 파일 삭제 및 1:1 질문 관련 TVC를 BaseQuestionTVC로 통일하는 작업을 수행했습니다.
  • tableView의 top inset 추가가 필요하여 해당 코드를 구현해주었습니다.
    • 첫번째 게시물만 top constraint이 16px이고, 나머지 게시물의 top constraint는 12 px여서 4px의 top inset이 필요했습니다.
  • 커밋은 f700ff9 부터 봐주세요!

📸 ScreenShot

jane1choi and others added 30 commits October 9, 2022 01:18
@jane1choi
➕ Add: 프로필 수정 버튼 추가 (#509)
7ec3b69
@jane1choi
✅ Chore: 프로필 수정 뷰 기본 UI 수정사항 반영 (#509)
b9d3237
@jane1choi
✨ Feat: 프로필 수정 한줄소개 글자수 표시 기능 구현 (#509)
ad050b5
@jane1choi
✅ Chore: 바텀시트 높이 커스텀 가능하도록 코드 수정 (#509)
93767e1
@jane1choi
✨ Feat: 프로필 이미지 변경 바텀시트 디폴트 UI 구성 (#509)
d0d5f80
@jane1choi
✅ Chore: 제2전공 선택 시 미진입 옵션 처리 위한 코드 수정 (#509)
a5d3a04
@jane1choi
✨ Feat: 바텀시트 호출 프로필 수정 뷰, 학과 리스트로 분기처리 (#509)
85c9a19
@jane1choi
⏪️ Rename: @IBAction 네이밍 수정 (#509)
58e9fc2
@jane1choi
✅ Chore: 컨벤션에 맞도록 함수 위치 조정 (#509)
80a12c8
@jane1choi
✅ Chore: univID에 맞는 학과무관 majorID 보내도록 수정 (#509)
10606e6
@dev-jungbin
✅ Chore: 홈탭 엑세스토큰 갱신 처리 (#509)
98ab550
@dev-jungbin
✅ Chore: 마이페이지탭 엑세스토큰 갱신 처리 (#509)
06c1529
@dev-jungbin
✅ Chore: 알림탭 엑세스토큰 갱신 처리 (#509)
49e0181
@dev-jungbin
Revert "✅ Chore: 알림탭 엑세스토큰 갱신 처리 (#509)"
2fe293f 
This reverts commit 49e0181.
@dev-jungbin
Revert "✅ Chore: 마이페이지탭 엑세스토큰 갱신 처리 (#509)"
ade5916 
This reverts commit 06c1529.
@dev-jungbin
Revert "✅ Chore: 홈탭 엑세스토큰 갱신 처리 (#509)"
733ed78 
This reverts commit 98ab550.
@jane1choi
➕ Add: 이미지 에셋 추가 (#509)
687fac7
@jane1choi
✅ Chore: 함수 문서화 주석 수정 (#509)
9ca2dcf
@jane1choi
🚚 Move: 이미지 에셋 폴더링 수정 (#509)
2eeef96
@hwangJi-dev
➕ Add: NadoSwitch UIComponent 구현 #524)
d4230b1
@jane1choi
✅ Chore: 프로필 수정 뷰 디폴트 UI 수정 (#509)
82a1b0d
@jane1choi
✅ Chore: 선택완료 버튼 조건 설정 (#509)
860aa81
@jane1choi
✨ Feat: 버튼 액션 설정 및 데이터 전달 구현 (#509)
ad1362a
@jane1choi
Merge remote-tracking branch 'origin/chore/#509-UI-Mypage-EditProfile…
c139720 
…' into chore/#509-UI-Mypage-EditProfile
@jane1choi
✅ Chore: 이미지 변경 및 리뷰반영한 코드 수정 (#509)
e9e7efe
@hwangJi-dev
✅ Chore: Classroom API 관련 코드 수정 (#524)
d06ab48
@hwangJi-dev
✅ Chore: 질문가능 선배 default값을 '선배 모두 보기'로 설정하도록 코드 수정 (#524)
7fd6fd5
@hwangJi-dev
✅ Chore: NadoSwitch 메서드 추가 및 코드 수정 (#524)
a1b661e
@hwangJi-dev
✅ Chore: QuestionPeopleHeaderView 레이아웃 수정 및 코드 추가 (#524)
a5d3527
@hwangJi-dev
✅ Chore: collectionView 레이아웃 코드 수정 및 API 코드 수정 (#524)
ccf70f0
jane1choi
jane1choi approved these changes Oct 17, 2022
View reviewed changes
Copy link
Member

@jane1choi jane1choi left a comment

Choose a reason for hiding this comment

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

확인했습니다! 고생했어요~!

@hwangJi-dev
✅ Chore: MajorIDConstants 학교별로 대응되도록 코드 수정 및 UnivType 추가 (#541)
9c19bff
@hwangJi-dev
➕ Add: communityFilter의 정보를 전달하는 SendCommunityFilterInfoDelegate 추가 (#…
21912a7 
…541)
@hwangJi-dev
➕ Add: MajorIDConstants 추가 (#541)
1b0c490
@hwangJi-dev
✨ Feat: HalfModalVC에 커뮤니티 필터기능 추가 (#541)
5c2ec21
@hwangJi-dev
✅ Chore: HalfModalVC 관련 코드 수정 (#541)
abca9b1
@hwangJi-dev
✅ Chore: CommunityWriteVC 관련 프로토콜 SendModalUpdateDelegate와 분리 (#541)
c18da26
@hwangJi-dev
✅ Chore: CommunityTVC UI 코드 수정 및 public 메서드 구현 (#541)
b6556b7
@hwangJi-dev
✨ Feat: 커뮤니티 필터기능 구현 (VC, Reactor) (#541)
e2fba48
@hwangJi-dev
✅ Chore: CommunityTVC 관련 코드 수정 (#541)
fc7348f
@hwangJi-dev
✅ Chore: 코드리뷰 반영하여 코드 수정 (#541)
31a8281
@hwangJi-dev
✅ Chore: 프로젝트 파일 수정 (#541)
f3ccbbe
@hwangJi-dev
✅ Chore: 파일 추가에 따른 프로젝트 파일 수정 (#541)
a11f6cb
@hwangJi-dev
✅ Chore: MajorIDConstants 학교별로 대응되도록 코드 수정 및 UnivType 추가 (#541)
55dfbb8
@hwangJi-dev
✅ Chore: 바텀시트 Community 관련 Delegate 이름 변경 및 적용 (#541)
6679c77
@hwangJi-dev
✅ Chore: 코드 수정 (#541)
b12f224
@hwangJi-dev
✅ Chore: develop 브랜치와 Merge (#541)
bfdb1e6
@hwangJi-dev
Merge pull request #543 from TeamNado-Sunbae/feature/#541-community-m…
343fbcd 
…ajorFilter

✨ [FEAT] 커뮤니티 학과 필터기능을 구현합니다.
@hwangJi-dev
⚰️ Remove: 필요하지 않은 파일 삭제 (#542)
90cbab1
@hwangJi-dev
🚚 Move: 파일 이동 (#542)
30d6712
@hwangJi-dev
✅ Chore: 초기화 단계에서 BaseQuestionTVC의 UI 구현부 호출을 제거하고 함수로 분리 및 파일 위치 이동 (#…
f32217f 
…542)
@hwangJi-dev
✅ Chore: 1:1 질문 관련 TVC 코드들 BaseQuestionTVC로 통일 (#542)
c5bb58c
@hwangJi-dev
✅ Chore: 기존 TVC를 EntireQuestionTVC로 사용하던 TableView에 top inset 추가 (#542)
2432cc1
@hwangJi-dev
✅ Chore: Rebase 후 없어진 파일 추가 및 코드 수정 (#542)
b540b8c
@hwangJi-dev
Merge remote-tracking branch 'origin/chore/#542-baseQuestionTVC-remov…
74b6c8e 
…eDuplicate' into chore/#542-baseQuestionTVC-removeDuplicate
@hwangJi-dev
✅ Chore: 머지 confilct 해결 (#542)
581a8a4
@hwangJi-dev hwangJi-dev merged commit 2f75ef2 into develop Oct 19, 2022
@hwangJi-dev hwangJi-dev deleted the chore/#542-baseQuestionTVC-removeDuplicate branch October 19, 2022 09:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jane1choi jane1choi jane1choi approved these changes

@dev-jungbin dev-jungbin dev-jungbin approved these changes

Assignees

@hwangJi-dev hwangJi-dev

Labels
✅ Chore 자잘한 수정 시 사용합니다. ✨🍕 Jieun 🐰✨ 지은이 연 pr, issue에 사용 💡 Refactor 코드 리펙토링 시 사용합니다.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

✅ [CHORE] BaseQuestionTVC관련 중복 파일을 제거하고 TVC를 통일합니다.
3 participants
@hwangJi-dev @dev-jungbin @jane1choi