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

Migrating to jetpack compose #44

Merged
merged 34 commits into from
Jan 5, 2022
Merged

Migrating to jetpack compose #44

merged 34 commits into from
Jan 5, 2022

Conversation

cmorigaki
Copy link
Owner

Context

Code

  • Changed the view state objects to a sealed class
  • Created a lot and probably not using the best practices with compose
  • Switched Fresco to Coil image library

Additional notes

  • The screens are a little different from the older implementation

@cmorigaki cmorigaki self-assigned this Jan 5, 2022
@cmorigaki cmorigaki merged commit c6961f6 into main Jan 5, 2022
@cmorigaki cmorigaki deleted the migrating-jetpack-compose branch January 5, 2022 22:35
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.

1 participant