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

Switch Compose Multiplatform #715

Merged
merged 2 commits into from
Aug 22, 2024
Merged

Conversation

shin-usu
Copy link
Contributor

Issue

Overview (Required)

I added logic to switch Compose Multiplatform.
For now, Timetable and TimetaleDetail screen of Compose Multiplatform only work. If you transition to other screen, app is crashed.

Links

Movie (Optional)

Before After
2024-08-22.1.23.47.mov

@shin-usu shin-usu marked this pull request as ready for review August 21, 2024 16:30
Comment on lines +102 to +112
private var delegateReducer: some ReducerOf<Self> {
Reduce { state, action in
switch action {
case .about(.delegate(.switchComposeMode)):
state.viewType = .compose
return .none
default:
return .none
}
}
.forEach(\.paths.about, action: \.paths.about)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I created new Reducer for delegate actions, but I don't know this is best practice.
If you have any ideas, please let me know🙏🏻

Copy link
Member

@ry-itto ry-itto left a comment

Choose a reason for hiding this comment

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

awesome! lgtm👍

@shin-usu shin-usu merged commit 1a4c2c9 into main Aug 22, 2024
7 checks passed
@shin-usu shin-usu deleted the feature/switch-compose-multiplatform branch August 22, 2024 00:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Switch app view to ComposeMultiplatform view from SwiftUI
2 participants