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

[refactor] 위니 피드 / 일회성 이벤트는 SharedFlow 사용하도록 변경 #300

Merged
merged 4 commits into from
Jun 26, 2024

Conversation

leeeha
Copy link
Member

@leeeha leeeha commented Jun 19, 2024

📝 Work Description

  • 기존에는 위니피드 삭제 성공 이벤트를 StateFlow로 수신하고 있어서, 상세피드 들어갔다가 나올 때마다 매번 스낵바가 반복적으로 뜨는 문제가 있었습니다. (이유: StateFlow가 피드 삭제 UiState의 최신 값을 구독하고 있어서)
  • 가장 최신 데이터를 구독하여 UI 상태를 갱신할 때는 StateFlow, 일회성 이벤트를 처리할 때는 SharedFlow를 사용해야 한다는 걸 뒤늦게 알게 되어서 위니피드 삭제 성공 이벤트는 SharedFlow를 사용하도록 변경했습니다.

📣 To Reviewers

  • 이번주 내로 리뷰 부탁드립니다!

@leeeha leeeha added 하은 🐰 refactor 🧹 코드 리팩토링 labels Jun 19, 2024
@leeeha leeeha requested a review from sxunea June 19, 2024 13:47
@leeeha leeeha self-assigned this Jun 19, 2024
@leeeha leeeha changed the title [refactor] 위니 피드 / 일회성 이벤트는 shared flow 사용하도록 변경 [refactor] 위니 피드 / 일회성 이벤트는 SharedFlow 사용하도록 변경 Jun 19, 2024
Copy link
Contributor

@sxunea sxunea left a comment

Choose a reason for hiding this comment

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

이제 stateflow 값 다시 초기화해줄일이 없겠네요 ! ! 👍🏻 덕분에 sharedflow 복습하고 갑니당 PR 본 김에 livedata, flow, shareflow, stateflow 차이 다시 봤는데 이 영상이 정리가 잘되어있더라구요 ! 우리는 세로모드 고정인데 가로로 회전할때 (구성변경) 일어나면 어떤 차이가 있는지도 알려주어서 좋았습니다 고생했어요 ❤️

@leeeha
Copy link
Member Author

leeeha commented Jun 26, 2024

영상 공유해줘서 고맙습니당 🫶

@leeeha leeeha merged commit 8b5837c into develop Jun 26, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactor 🧹 코드 리팩토링 하은 🐰
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[refactor] 위니 피드 / 일회성 이벤트는 SharedFlow 사용하도록 변경
2 participants