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

[fix] 상세 피드 / 댓글 생성, 삭제 후 empty 뷰 전환하기 #146

Merged
merged 6 commits into from
Aug 23, 2023

Conversation

leeeha
Copy link
Member

@leeeha leeeha commented Aug 23, 2023

📝 Work Description

  • 리사이클러뷰에 댓글 생성 시, 현재 리스트가 비어있는지 확인 -> ListAdapter의 currentList 체크
  • 리사이클러뷰로부터 댓글 삭제 후, 현재 리스트가 비어있는지 확인 -> deleteItem 함수에서 반환하는 newList의 사이즈 체크

📣 To Reviewers

  • 오늘 7시 QA 전에는 머지할 예정입니다!
  • ListAdapter 자체의 currentList (immutable list) 와 addItem, deleteItem 함수의 newList (mutable list) 가 반환하는 아이템 개수가 불일치 하기 때문에 위와 같은 방식으로 구현했습니다..! 더 좋은 방법이 있다면 제안 부탁드려요 🙏

@leeeha leeeha added fix 💥 버그 및 오류 해결 하은 🐰 labels Aug 23, 2023
@leeeha leeeha self-assigned this Aug 23, 2023
Copy link
Contributor

@Sangwook123 Sangwook123 left a comment

Choose a reason for hiding this comment

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

👍👍👍👍

@leeeha leeeha merged commit f96c88b into develop Aug 23, 2023
1 check passed
@leeeha leeeha deleted the feature/fix-comment-empty-switch branch August 24, 2023 12:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix 💥 버그 및 오류 해결 하은 🐰
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[fix] 상세 피드 / 댓글 생성, 삭제 시 empty 뷰 전환하기
2 participants