Follow-up from "Design changes for News feeds Page"
The following discussion from !112 should be addressed:
-
@beit--tu-chemnitz.de started a discussion: Würde den Code so mergen, das Verhalten müssen wir in der nächsten Version ändern. Eigentlich muss das Verhalten folgendermaßen Aussehen:
- View lädt alle Feeds und Feeditems aus dem Cache
- View öffnet sich und zeigt die Feedtabs und die Feeditems des ausgewählten Feeds/Feedtabs. Hier werden nur Daten aus dem Cache verwendet.
- View holt sich liste aller Feeds und aktualisiert die Feedtabs und den Cache.
- View holt sich Feeditems des aktuellen Feeds und aktualisiert Feeditems des angezeigten Feeds/Feedtabs
Wichtig ist dabei das auf dem Lifecycle der React-Komponenten Rücksicht genommen wird: https://reactjs.org/docs/state-and-lifecycle.html#adding-lifecycle-methods-to-a-class Vorschlag:
- View lädt alle Feeds und Feeditems aus dem Cache -> Konstruktor
- View öffnet sich und zeigt die Feedtabs und die Feeditems des ausgewählten Feeds/Feedtabs. Hier werden nur Daten aus dem Cache verwendet. -> render()
- View holt sich liste aller Feeds und aktualisiert die Feedtabs und den Cache. -> componentDidMount()
- View holt sich Feeditems des aktuellen Feeds und aktualisiert Feeditems des angezeigten Feeds/Feedtabs und den Cache -> componentDidMount()
Wobei ich glaube das durch Redux ein update des Caches auch die Views aktualisiert.