Hinzufügen des Supports für Flat Tarife im Vodafone Widget

[Chaos53925]

Zur Zeit werden im Vodafone Widget die Tarife, die eine Datenvolumen Flat haben mit 100000 GB Datenvolumen angezeigt.

Dieser Pull request beinhaltet änderungen, die genau das ändern und das entsprechend anzeigen.

Bei weiteren Fragen gerne melden.

Zur info: Das ist mein erster Pull request also bitte nicht wundern, das da soviele aktionen passiert sind und es ein wenig wirr wirkt. Musste erstmal verstehen wie das alles geht.

[ThisIsBenny]

Hi, leider ist es nicht so leicht nur die nötigen Änderung für die Unlimited Tarife zu sehen. Wird etwas Zeit dauern und wahrscheinlich werde ich es etwas umschreiben, da mir bereits ein paar Punkte aufgefallen sind:

• Es wird an mehreren Stellen geprüft ob es eine Flat ist. Dies sollte jedoch nur an einer Stelle gemacht werden
• die erstelle Diagramm funktioniert wurde so umgeschrieben, dass sie im Fall von Unlimited den Inhalt austauscht. Man sollte aber bessere eine andere Funktion verwenden im Fall von unlimited um sich Schritte die für unlimited nicht nötigen sind zu überspringen, auch der die Widget Family ist global verfügbar, weshalb man sie nicht übergeben muss

Ich hatte mir den Pull Request bisher nur am iPhone angeschaut, weshalb ich bisher keine anderen Änderungen erkannt habe. Ist ggfs. Noch etwas anderes nötige als zu prüfen ob es unlimited ist und entsprechend ein anderes Diagramm anzuzeigen.

An sich stelle ich mir eh die Frage warum man ein unlimited Tarif unterstützen sollte. Ändern tut sich an der Anzeige doch eh nicht oder?

[Chaos53925]

Die Teile mit Unlimited Tarife haben bei mir immer ein Kontingent von 100000 Gb angezeigt. Bei den Giga-Pass für Streaming Musik und den anderen war es das selbe.

Natürlich kann man das anders lösen, aber das ist der Weg der mir in den Sinn gekommen ist und auf dem iPhone funktionierte. Hat mich nur 3-4 Stunden gekostet.

Die Änderungen sind am Ende eher klein und wahrscheinlich eher was für Autisten wie mich, aber so kleine Spielereien haben m.M.n. auch ihren reiz. ;)

[ThisIsBenny]

Problem ist, dass dein Code Editior sehr viel mehr geändert hat, weshalb er mit 600 Änderungen anzeigt.

Wenn ich es richtig sehe sind es aber wahrscheinlich nur ca. 30 Zeilen Code änderungen.
Hast du die Möglichkeit den Pull Request so zu stellen, dass nur diese 30 Zeilen Code änderung drin sind?
Falls nicht, würde ich im meinem Repo ein neuen Branch aufmachen, dort die gewünschten Änderungen durchführen und dir den einen Pull Request für ein Review zu schicken, um sicher zu gehen, dass deine ganzen gewünschten Änderungen drin sind.

[Chaos53925]

Wenn es für dich in Ordnung ist würde ich das dann gerne morgen versuchen (auch wenn ich es heute wahrscheinlich heute ebenfalls versuchen werde).
Wie die 600+ Änderungen zustande kommen ist selbst mir unklar, da ich Visual Studio Code verwendet habe und da bei meinem anderm Projekt nie so abgedreht hat.

Vielleicht hilft es einen neuen Fork zu erstellen und von dort aus einen neuen Pull zu erstellen.
Eine Garantie habe darauf aber nicht.
Zuerst werde ich habe schauen ob es durch den revert aller veränderungen eine besserungen geht.
Ich werde trotzdem mal meine Konfiguration durchschauen um zu sehen was da falsch läuft.

[ThisIsBenny]

Du musst in Visual Studio Code das automatische Formatieren ausschalten (unten rechts in der Statusbar).
Und ein neuer Fork wird auch nötig sein, da du es wahrscheinlich nicht mehr sauber trennen kannst.

[Chaos53925]

Stand jetzt liegt die Anzeige bei +44 -16

Das sieht schon viel besser aus und es ist nun sicher einfacher die änderungen zu sehen.
Der revert aller änderungen hat es auf jeden Fall gebracht.
Die Konfiguration habe ich auch zurückgesetzt und neu eingerichtet. Ich hoffe, dass das für divh so besser zu erkennen ist welche änderungen ich gemacht habe um die Anzeige so hinzubekommen wie ich mir das vorgestellt habe.

Natürlich bin ich für weitere Vorschläge offen, aber erwarte von mir bitte nicht all zu viel. ;)

[Chaos53925]

Habe mir nochmal ein paar gedanken gemacht und nun stehe ich bei +39 -17 an änderungen.