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.

Sign up for GitHub

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

Jump to bottom

(Issue #1059) WorkManagerSample to use Hilt and App Startup #1060

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

(Issue #1059) WorkManagerSample to use Hilt and App Startup #1060

wants to merge 2 commits into from
+155 −32

Conversation

DivS-15
Copy link

@DivS-15 DivS-15 commented Jul 15, 2022 •
edited

closes #1059
Following changes were made:

  1. WorkManager instance is initialized on-demand using the app-startup library and injected in the FilterViewModel using Hilt and @Provides method.
  2. Workers now use @HiltWorker along with Assisted injection.
  3. Complete app uses Hilt for dependency injection.

@DivS-15
Workers use @HiltWorker Assisted injection
524f2d5
@DivS-15
Used App startup with Hilt to provide work-manager instance on demand
ff5b093 
Refactored the whole app to use Hilt for dependency injection, especially the worker classes.
@DivS-15
Copy link
Author

DivS-15 commented Aug 18, 2022

@dlam @ianhanniballake Could you please look into this? This might improve the sample.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

'WorkManagerSample' should use Hilt and App Startup library to initialize and provide WorkManager dependency
1 participant
@DivS-15