Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Cache für Kartendaten #91

Merged
merged 1 commit into from
Nov 1, 2023
Merged

Cache für Kartendaten #91

merged 1 commit into from
Nov 1, 2023

Conversation

svendroid
Copy link
Contributor

@svendroid svendroid commented Oct 31, 2023

Solves #76

Fügt einen Cache für die Kartendaten hinzu. Die Kartendaten werden beim ersten Aufrufen der Kartenabschnitte heruntergeladen und im Anschluss auf dem Gerät 30 Tage gecached. Das sollte den Datenverbrauch senken. Den Zeitraum von 30 Tagen kann man beliebig einstellen, hab mich an dem im Ticket vorgeschlagenen Monat orientiert.

Man kann den Kartencache in den Einstellungen auch manuell löschen, wenn man will, sollte aber eigentlich nur zum Testen notwendig sein.

simulator_screenshot_ABB308DB-A1A1-4101-A15C-AAE6A130F9DF

Was meinst Du? @thomas-munichways

@svendroid svendroid changed the title Cache für Kartendaten #76 Cache für Kartendaten 76 Oct 31, 2023
@svendroid svendroid changed the title Cache für Kartendaten 76 Cache für Kartendaten #76 Oct 31, 2023
@svendroid svendroid changed the title Cache für Kartendaten #76 Cache für Kartendaten Oct 31, 2023
@svendroid svendroid merged commit 9721d3f into master Nov 1, 2023
1 check passed
@svendroid svendroid deleted the 76_add_map_cache branch November 1, 2023 16:41
@uvelogs
Copy link
Contributor

uvelogs commented Nov 1, 2023

@svendroid , @thomas-munichways
Servus Sven, Servus Thomas,
ich bin gerade dabei, einen ProgressBar zu integrieren. Wenn "Radnetz neu laden" ausgeführt wird, erscheint ein ProgressBar und verschwindet dann wieder. Es handelt sich um Issue #60.

ProgressBar

Da nun eine weiterere Einstellung dazu gekommen ist, wäre es evtl. besser noch eine Abfrage einzubauen, ob man direkt zur Karte navigieren oder auf der Einstellungs Seite verbleiben möchte.

Was meint Ihr dazu?

@svendroid
Copy link
Contributor Author

svendroid commented Nov 1, 2023

Ich finde die Idee gut den Ladezustand auf der Einstellungsseite anzuzeigen.
Statt dem Progressbar fände ich ein Ladekringel gut, der das Icon ersetzt gut, dann ist auch klar, dass man nicht nochmal klicken kann bzw. das etwas passiert. Das "Mülleimer" Icon könnte man dann auch durch ein Downloadicon ersetzen, dann wird klarer, dass nicht nur gelöscht wird. Evtl. könnte man auch an der Stelle anzeigen, welche Version gerade geladen ist oder wann sie das letzte Mal aktualisiert wurde, dann ist das auch deutlich.

Bzgl. direkter Navigation zur Karte finde ich persönlich das etwas verwirrend, weil das ja eigentlich nix mit dem Neuladen des Radnetzes zu tun hat und ein Nutzer nicht vermutet, dass man auf einen anderen Screen springt. Und der wäre hier ja nur ein Back-klick entfernt. So zumindest meine Annahme.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants