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] HomeState 초기화 로직 수정 - merge into #76 #82

Merged
merged 2 commits into from
Feb 1, 2025

Conversation

jayden000106
Copy link
Contributor

@jayden000106 jayden000106 commented Jan 31, 2025

📌 Summary

지난 회의 때 이야기 했던 것처럼 HomeState를 초기화해줄 때 실제 값을 받아와서 만들어 주는 방식으로 수정

✍️ Description

💡 PR Point

튜플을 통해서 HomeReducer.State를 만드는 부분은 추후 수정이 필요하지만, 의도했던 부분은 회의 중 전달 완료

📚 Reference

🔥 Test

- 앱 초기화 시에 TotalReportCard 및 StudentInfo를 초기화하도록 수정
- HomeView에서 OnAppear에서 Init하지 않도록 수정
@jayden000106 jayden000106 self-assigned this Jan 31, 2025
@jayden000106 jayden000106 added the refactor🪄 리팩토링 label Jan 31, 2025
@jayden000106 jayden000106 requested a review from CJiu01 January 31, 2025 11:50
@jayden000106 jayden000106 changed the title [#76-Jayden] HomeState 초기화 로직 수정 [Refactor] HomeState 초기화 로직 수정 - merge into #76 Jan 31, 2025
@LeeJoEun-01 LeeJoEun-01 merged commit d017f51 into refactor/#76 Feb 1, 2025
@LeeJoEun-01 LeeJoEun-01 deleted the refactor/#76-jayden branch February 1, 2025 15:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactor🪄 리팩토링
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants