Skip to content

Fix [#55] 소식, 알림 탭 오류 및 로직 수정 #56

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
Merged
JinUng41 merged 11 commits into from
Nov 12, 2024
Merged

Fix [#55] 소식, 알림 탭 오류 및 로직 수정 #56

JinUng41 merged 11 commits into from
Nov 12, 2024

Conversation

@JinUng41
Copy link
Collaborator

@JinUng41 JinUng41 commented Nov 11, 2024 •
edited
Loading

👻 PULL REQUEST

🛠️ PR POINT

  • [Fix] 소식, 알림 탭 오류 수정 #55 에서 설명하는 오류 3가지 중 2가지 해결 (오늘 경기,, + 인기글 관련)
  • ViewModel과 ViewController 간 바인딩 로직 수정 (ViewModelType)
  • UITableViewDataSource -> DiffableDataSource
  • gitignore에 GoogleService-Info.plist 추가
  • CancelBag 로직 수정
  • MoyaLoggingPlugin 수정
  • ReuseIdentifiable 추가

💡 참고사항

커밋 단위가 무너져서 파일 단위로 봐주세요.
프로필 이미지 터치 시 동작은 기반 작업은 완료하였으나, 몇 가지 확인 할 것이 있어 보류하였습니다.
알림-활동 페이지는 데이터가 없어 라이브 서버에서 확인하였습니다. 기기는 11 pro max 입니다.

📸 스크린샷

기능 스크린샷
오늘 경기가 있는 경우
오늘 경기가 없는 경우
인기글 관련

📟 관련 이슈

@JinUng41
[Docs] #55 - GoogleService-Info.plist gitignore에 추가
8671ea1
@JinUng41
[Add] #55 - ReuseIdentifiable 추가
f25cb73 
- 그 외 기타 파일 추가
@JinUng41
[Chore] #55 - 불필요한 코드, 빈 줄 삭제 및 기타 코드 수정
a29cf5b
@JinUng41
[Fix] #55 - CancelBag 소멸자에 구독 해제하도록 수정
977c080
@JinUng41
[Fix] #55 - 타 유저의 인기글 알림을 본인의 인기글로 표시하는 오류 수정
dad1040 
- 그 외 주석 제거 및 메서드 파라미터 네이밍 수정
@JinUng41
[Refactor] #55 - NotificationViewController 리팩토링
9f41b6a 
- 불필요한 메서드 삭제
- 함수 분리
- 프로토콜 확장 분리
@JinUng41
[Chore] #55 - 간단한 코드 수정 및 사용하지 않는 메서드 제거
2fd05ae
@JinUng41
[Refactor] #55 - 알림 탭 활동 로직 전면 수정
e7bd1b2 
- ViewModelType(Input-Output)을 따르도록 수정
- 뷰컨트롤러의 비즈니스 로직을 뷰모델로 이동
- DiffableDataSource 사용 (그에 따른 모델 Hashable 채택)
- 프로필 이미지 터치 시 일어나야 할 작업은 추후 커밋에서 할 예정
- 오타 수정
@JinUng41
[Fix] #55 - MoyaLoggingPlugin 수정
86406f2
@JinUng41
[Refactor] #55 - 소식 경기 탭 로직 전면 수정 및 오늘 경기 관련 오류 해결
f9b818e
@JinUng41
[Fix] #55 - 화면이 다시 보여질 때, 새 페이징이 불가한 현상 수정 및 주석 추가
50bfe37
@JinUng41 JinUng41 added 🛠️ fix 기능적 버그나 오류 해결 시 사용 🍻 진웅 술 한잔 가온나~ labels Nov 11, 2024
@JinUng41 JinUng41 requested a review from binisnil November 11, 2024 04:09
@JinUng41 JinUng41 self-assigned this Nov 11, 2024
binisnil
binisnil approved these changes Nov 11, 2024
View reviewed changes
Copy link
Collaborator

@binisnil binisnil left a comment

Choose a reason for hiding this comment

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

고생많으셨습니다!!

@JinUng41 JinUng41 merged commit 67202c5 into develop Nov 12, 2024
@JinUng41 JinUng41 deleted the fix/#55 branch November 12, 2024 07:53
@JinUng41 JinUng41 restored the fix/#55 branch November 12, 2024 07:53
@JinUng41 JinUng41 changed the title [Fix] #55 소식, 알림 탭 오류 및 로직 수정 Fix [#55] 소식, 알림 탭 오류 및 로직 수정 Nov 12, 2024
@youz2me youz2me deleted the fix/#55 branch February 28, 2025 13:58
youz2me pushed a commit that referenced this pull request Oct 26, 2025
@JinUng41
[Merge] #55 - pull request #56 from Team-Wable/fix/#55
ee62ac0 
[Fix] #55 소식, 알림 탭 오류 및 로직 수정
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@binisnil binisnil binisnil approved these changes

Assignees

@JinUng41 JinUng41

Labels

🍻 진웅 술 한잔 가온나~ 🛠️ fix 기능적 버그나 오류 해결 시 사용

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[Fix] 소식, 알림 탭 오류 수정

3 participants

@JinUng41 @binisnil