TeamNewPipe/NewPipe

CommentRepliesSource: Add cache for reply fetching

Open

#11.718 geöffnet am 18. Nov. 2024

Auf GitHub ansehen
 (2 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Java (3.577 Forks)batch import
bugcommentshelp wantedrewrite

Repository-Metriken

Stars
 (38.153 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 9T 17h) (17 gemergte PRs in 30 T)

Beschreibung

There is no cache in the comment replies fetching, causing the app to reload them each time the device is rotated or a switch between the description content view tab and the comments one is done. This regression must be fixed.

Originally posted by @AudricV in https://github.com/TeamNewPipe/NewPipe/pull/11060#discussion_r1836886664

Contributor Guide

Research-Richtung
Untersuche die aktuelle Logik zum Abrufen von Antworten und implementiere eine Caching Schicht (z.B. In Memory Cache oder Shared Preferences), um das erneute Laden von Daten bei Drehung oder Tab Wechsel zu vermeiden.
Tech Stack
java
Domain
mobile
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Frisch
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Meist klar
Voraussetzungen
JavaAndroid
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70