TeamNewPipe/NewPipe

CommentRepliesSource: Add cache for reply fetching

Aperta

#11.718 aperta il 18 nov 2024

Vedi su GitHubGuida contributor
 (2 commenti) (0 reazioni) (0 assegnatari)Java (3577 fork)batch import
bugcommentshelp wantedrewrite

Metriche repository

Star
 (38.153 stelle)
Metriche merge PR
 (Merge medio 9g 17h) (17 PR mergiate in 30 g)

Descrizione

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

Guida contributor

Direzione di ricerca
Analizza la logica corrente di recupero delle risposte e implementa un livello di cache (ad esempio, cache in memoria o preferenze condivise) per evitare di ricaricare i dati durante la rotazione o il cambio di scheda.
Tech stack
java
Dominio
mobile
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
JavaAndroid
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70