android / android/nowinandroid
[Bug]: Should "bindsUserNewsResourceRepository" be moved to "DataModule"?
- Dominant language
- Kotlin
- Stars
- 21.8k
- Forks
- 4.6k
- Avg merge
- 19h 20m
- Merged PRs (30d)
- 2
Description
### Is there an existing issue for this?
- [X] I have searched the existing issues
### Is there a StackOverflow question about this issue?
- [X] I have searched StackOverflow
### What happened?
There is already interface `DataModule`, should abstract method `bindsUserNewsResourceRepository` be written in interface `DataModule`?
### Relevant logcat output
_No response_
### Code of Conduct
- [X] I agree to follow this project's Code of Conduct
Contributor guide
Research direction
Start by reading the DataModule interface and the bindsUserNewsResourceRepository declaration, then trace their implementations and usages. Determine whether the method belongs in the interface and verify that affected implementations and callers remain consistent; the issue is done when the architectural question is resolved and the necessary references are updated.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- kotlin
- Domain
- mobile
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 25/100