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

Revisit DI #20

Closed
physphil opened this issue Jul 30, 2019 · 0 comments
Closed

Revisit DI #20

physphil opened this issue Jul 30, 2019 · 0 comments

Comments

@physphil
Copy link
Owner

Remove Dagger, attempt with just constructor injection.
physphil added a commit that referenced this issue Aug 22, 2019
@physphil
GH-20: remove Dagger
fcba60c 
* create Injector
* create method for creating DB
* remove Dagger DI from ShowNotificationJob and SnoozeBroadcastReceiver
physphil added a commit that referenced this issue Aug 22, 2019
@physphil
GH-20: remove Dagger from ReminderViewModel
3051b8a
physphil added a commit that referenced this issue Aug 22, 2019
@physphil
GH-20: remove Dagger from MainActivity
3a5a3af
physphil added a commit that referenced this issue Aug 22, 2019
@physphil
GH-20: remove Dagger and components
8324cb4
physphil added a commit that referenced this issue Aug 22, 2019
@physphil
Merge pull request #33 from physphil/feature/GH-20_remove_dagger
906b96c 
GH-20: Remove Dagger
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@physphil