pockethub/PocketHub

Architecture

Open

#1.160 geöffnet am 23. Feb. 2018

Auf GitHub ansehen
 (7 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Java (3.521 Forks)batch import
discussionhelp wanted

Repository-Metriken

Stars
 (9.401 Stars)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

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-Richtung
Untersuche die aktuelle Codebase Struktur, mit Fokus auf IssueFragment und andere große Dateien, die UI und Netzwerkcode vermischen. Überprüfe die Kommentare im Issue für Vorschläge. Schlage ein sauberes Architekturmuster (z. B. MVP oder MVVM) vor, um die Zuständigkeiten zu trennen. Betrachte, wie Netzwerkaufrufe behandelt werden, und schlage einen Refactoring Plan vor.
Tech Stack
javaandroid
Domain
mobile
Issue Type
Refactoring
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Halber Tag
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Recherche nötig
Voraussetzungen
AndroidJavaArchitecture patterns
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
30