pockethub/PocketHub

Architecture

Open

#1,160 opened on Feb 23, 2018

View on GitHub
 (7 comments) (0 reactions) (0 assignees)Java (3,521 forks)batch import
discussionhelp wanted

Repository metrics

Stars
 (9,401 stars)
PR merge metrics
 (No merged PRs in 30d)

Description

As it stands the app mixes a lot of UI and network code. I makes it hard to understand the flow and to see where to UI changes are made. Other problems include huge files, IssueFragment is a good example.

I'm open to suggestions to how we can fix this.

Contributor guide

Research direction
Propose architectural improvements to separate UI and network code, evaluating patterns like MVP, MVVM, or Clean Architecture; consider dividing large files (e.g., IssueFragment) into smaller components.
Tech stack
javaandroid
Domain
mobile
Issue type
Refactor
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
3
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
Half day
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Active
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Needs investigation
Prerequisites
AndroidJavaArchitecture patterns
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
30