Skip to content

[Fix] #109 - 수정하기 순환 참조 문제 해결 & 저장 버튼 활성화 로직 수정#110

Merged
L-j-h-c merged 2 commits intoTeamRecorDream:developfrom
L-j-h-c:fix/#109-modifyFix
Jan 8, 2023
Merged

[Fix] #109 - 수정하기 순환 참조 문제 해결 & 저장 버튼 활성화 로직 수정#110
L-j-h-c merged 2 commits intoTeamRecorDream:developfrom
L-j-h-c:fix/#109-modifyFix

Conversation

@L-j-h-c
Copy link
Copy Markdown
Contributor

@L-j-h-c L-j-h-c commented Jan 8, 2023

👻 작업한 내용

수정하기 뷰에서 순환 참조로 인해 기존 셀이 메모리에서 해제되지 않던 문제를 해결했습니다.
수정하기 API가 정상 작동하게 수정했습니다.

📮 관련 이슈

@L-j-h-c L-j-h-c added the 록시보이 🌽 담당자 label Jan 8, 2023
@L-j-h-c L-j-h-c self-assigned this Jan 8, 2023
Copy link
Copy Markdown
Member

@EunHee-Jeong EunHee-Jeong left a comment

Choose a reason for hiding this comment

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

LGTM~~!!

override func prepareForReuse() {
super.prepareForReuse()
self.disposeBag = DisposeBag()
}
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

굿굿

Comment on lines -265 to +275
dataSource = UICollectionViewDiffableDataSource<Section, AnyHashable>(collectionView: dreamWriteCollectionView, cellProvider: { collectionView, indexPath, itemIdentifier in
dataSource = UICollectionViewDiffableDataSource<Section, AnyHashable>(collectionView: dreamWriteCollectionView, cellProvider: { [weak self] collectionView, indexPath, itemIdentifier in
guard let self = self else { return UICollectionViewCell() }
switch Section.type(indexPath.section) {
case .main:
self.mainCell = nil
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

굿굿

Copy link
Copy Markdown
Member

@Suyeon9911 Suyeon9911 left a comment

Choose a reason for hiding this comment

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

수고하셨습니당 ~~!

@L-j-h-c L-j-h-c merged commit c4d9159 into TeamRecorDream:develop Jan 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Fix] 수정하기 순환 참조 문제 수정

3 participants