Skip to content

Conversation

@manuelvicnt
Copy link
Contributor

Changes made:

  • Introduces a basic data layer in the app Repository and DataSource classes.
  • Uses Hilt as the DI solution to manage dependencies. It works perfectly because the viewModel() function used in Compose code gets the default ViewModelFactory created by Hilt with the @ViewModelInject annotation.
  • Tests also use Hilt as now we heavily rely on ViewModels in our composables.

@chrisbanes chrisbanes mentioned this pull request Oct 26, 2020
@manuelvicnt manuelvicnt merged commit 91c38e5 into dev_alpha06 Oct 27, 2020
@manuelvicnt manuelvicnt deleted the mv/crane_hilt branch October 27, 2020 13:10
chrisbanes added a commit that referenced this pull request Oct 28, 2020
* [Jetsnack] Cart implementation
* [Jetsnack] Reuse Quantity Selector
* [Jetsnack] Reuse Destination Bar
* [Jetsnack] Add Inspired By Section to Cart
* [Jetsnack] Add Checkout Bar to cart
* [Jetsnack] Fix Spotless issues
* [Jetsnack] Fix review comments
* [Jetsnack] Fix review comments
* Update to snapshot 6922857.
* [Jetsurvey] Bump to alpha06
* [Owl] Update to snapshot 6922857.
* [JetChat] Update to snapshot 6922857
* [Rally] Update to snapshot 6922857
* [Jetcaster] Update to snapshot 6922857.
* Removes unncessary elevation param
* [Crane] Update to 6922857 snapshot (#246)
* [Owl] Replace elevation overlay workaround with AmbientElevationOverlay.
* [Jetchat] Replace elevation overlay workaround with AmbientElevationOverlay
* [Jetsurvey] Redesigned next/prev buttons
* [Jetsurvey] Redesigned progress indicator
* [Jetsurvey] Adding elevation to the action bar
* [Jetsurvey] Capitalising strings
* [Jetsurvey] Question title redesigned
* [Jetsurvey] Removing unused composable
* [Jetsurvey] Removing the column usage in progress indicator
* [Crane] Adds Hilt to the project (#237)
* [JetNews] bump to alpha06 (#258)
* [JetNews] bump to alpha06
* Fix unused modifier on BookmarkButton
* Update all samples to AGP 4.2.0-alpha15
* Turn on allWarningsAsErrors in Jetsurvey
* Tidy up gradle.properties

We now have parallel compilation, caching
and on-demand configuration enabled on all projects

* Disable Jetifier where possible
Only Crane requires Jetifier becuase it uses the
Google Maps SDK.

* Update all projects to ktlint 0.39.0
* Update snapshot URL
* Update to Compose 1.0.0-alpha06

Co-authored-by: Jolanda Verhoef <[email protected]>
Co-authored-by: Nick Butcher <[email protected]>
Co-authored-by: Nick Butcher <[email protected]>
Co-authored-by: Florina Muntenescu <[email protected]>
Co-authored-by: Jose Alcerreca <[email protected]>
Co-authored-by: Jolanda Verhoef <[email protected]>
Co-authored-by: Jose Alcérreca <[email protected]>
Co-authored-by: Florina Muntenescu <[email protected]>
Co-authored-by: Manuel Vivo <[email protected]>
Co-authored-by: Sean McQuillan <[email protected]>
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.

3 participants