Anzeige der Sensorenkurven im Dashboard

Ich habe die Antwort auf dieses Rätsel, es ist ganz einfach: -1 soll den Wert « unbegrenzt » darstellen, das ist das, was im Code des Signups definiert ist, wenn du auf den Radio-Button « unbegrenzt » klickst.

Das Problem ist, dass es einen doppelten Bug im Code und in den Tests gab, wodurch dieser Wert -1 nie funktioniert hat…

Was passiert ist:

  • « -1 » hat sich so verhalten, als ob du nur die Zustände behalten wolltest, die älter sind als « in 1 Tag in der Zukunft » (NOW - (-1) = MORGEN) :scream: Es gab kein if(-1), um das Löschen der Zustände zu blockieren, ich denke, ich habe das beim Codieren vergessen. Da wir die vergangenen Sensorwerte noch nicht verwendet haben, ist es nicht aufgefallen. Was mich wundert, ist, dass mir scheint, dass @Terdious in seiner Instanz einen unbegrenzten Verlauf hat. Bestätigst du das @Terdious? Ich denke, du hast diese Variable absichtlich in der DB gelöscht, oder?
  • In den Tests, ein ziemlich dummer Fehler, ich habe fake.resolves() verwendet, um den Aufruf von variable.getValue() zu mocken, aber ich habe fake.resolves() falsch verwendet und deshalb hat sich der -1 bei meinen Tests damals so verhalten, wie gewünscht, denn im Grunde hatte ich eine zurückgegebene Funktion statt -1, und dadurch wurde das Löschen verhindert, kurz gesagt, etwas sehr Dummes (das war ganz am Anfang der v4, vor etwa eineinhalb Jahren, denke ich :D)

Ich habe den Bug behoben und zusätzliche, robustere Tests hinzugefügt, um sicherzustellen, dass sich nichts mehr ändert.

In der UI gibt es jetzt die Möglichkeit, den Wert in den Einstellungen zu ändern: