Cache data #59
Cache data #59
Conversation
+ Added "global" stream for loading data at the loading screen or at the screen of the feature.
+ Now everything else should be faster to refactor.
There was a problem hiding this comment.
Das sieht und fühlt sich erstmal sehr gut an.
Aber der "der Benutzer is Offline" fall ist glaub ich nicht genügend behandelt.
Wenn der Nutzer eine request offline macht, wärend er schon angemeldet wurde, dann kommen ein haufen fehler. Stattdessen wäre eine Snackbar "du bist offline" oder so hilfreich. Dann könnte man die daten, die schon geladen sind auch da lassen. (Ich weiß, dass ich das so auch bisher nicht wirklich realisiert habe...)
Und den daten zur liebe, würde ich "Nur vertretungsplan laden" als default einstellung nehmen.
|
Das heißt bei der Funktion fetchData() beim Fetcher muss ich einen Check hinzufügen ob der User offline ist und somit einfach kein request machen Zu Glück nicht schwer |
|
Beim offline sein würde ich bisschen penetranter sein, nämlich bei der App bar bottom property etwas hinzufügen oder über den ganzen Scaffold etwas anzeigen, aber dann funktioniert die ganze App nicht mehr |
Finde ich gut. |
|
hmmm merge war doch nicht so einfach |
|
glaube sollte gefixt sein |
|
Jetzt zu 100% ready |
|
bitte es sollte jetzt ready sein |
Hier pushen
Ladet (wenn man es will) am Anfang alles, cached es, ladet alle 15min neu (bei full mode).
Hoffentlich habe ich nix vergessen, etwas massiv die PR.
Code ist jetzt nicht wirklich der beste aber gut genug.
Gemachte ToDo:
Bei Main da wo alles gefetcht und zum fetcher hinzufügt wird wird kann man die Funktion zum fetchen (client.get..., fetcher.addData(...)) einfach gleich in den fetcher einbauen.