Kunzisoft / Kunzisoft/KeePassDX

Dependency Injection

Open
#2,558 0 comments 0 reactions 0 assignees View on GitHub
enhancement
Dominant language
Kotlin
Stars
7.3k
Forks
401
Avg merge
2d 10h
Merged PRs (30d)
1

Description

### Checks

- [x] I have read the Wiki, searched the open issues, and still think this is a new feature.

### Explain the problem clearly and succinctly:

_No response_

### Describe the solution you'd like:

Many ViewModels still use AndroidViewModel; the code should be migrated to Koin to inject resources and databases in order to decouple dependencies.

### Describe alternatives you've considered:

_No response_

### Additional context:

_No response_

Contributor guide

Open the contributing guide

Research direction

Start by locating the ViewModels that still extend AndroidViewModel and review how they access resources and databases. Define the migration scope around Koin injection and verify that those dependencies are decoupled from the ViewModels without changing their behavior.

Written by the indexing model from the issue text.

Assessment

Tech stack
android, kotlin
Domain
mobile
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Quiet
Clarity
Needs clarification
Newbie friendliness
30/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.