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

Statistik: GEX Belohnungen "seit Dienstag" ohne Dienstag #917

Closed
frank42hh opened this issue Apr 29, 2020 · 14 comments
Closed

Statistik: GEX Belohnungen "seit Dienstag" ohne Dienstag #917

frank42hh opened this issue Apr 29, 2020 · 14 comments
Labels
Bug Next Version Issue will be dealt with in the next update

Comments

@frank42hh
Copy link

Moin,
Erst mal eins vorweg: Ich habe mich unheimlich gefreut über das Statistik Modul, ich finde es klasse.
Leider scheint es noch ein paar Kinderkrankheiten zu haben. Was mir heute aufgefallen ist:
"Belohnungen -> GEX -> Seit Dienstag" beinhaltet "Dienstag" nicht.

Ich habe gestern Mittag, nach der Installation des Updates, die GEX gestartet. Ich habe gestern belohnungen erhalten, und die wurden gestern auch angezeigt.
Heute bin ich auf der Karte weitergegangen und habe weitere Belohnungen erhalten. Allerdings werden nur die heutigen angezeigt, nicht die von Gestern. Wenn ich auf "Belohnungen -> GEX -> letzten 7 Tage" gehe, werden die von gestern mit angezeigt.

GEX-Stats-2
GEX-Stats-1

@Gindi4711 Gindi4711 added Idea Feature or Change Requests Investigating and removed Idea Feature or Change Requests labels May 1, 2020
Gindi4711 added a commit that referenced this issue Jun 28, 2020
Fehlerbehebung Statistik (Issue #917)
@outoftheline outoftheline added Next Version Issue will be dealt with in the next update and removed Investigating labels Jun 29, 2020
@Gindi4711
Copy link
Collaborator

Fixed in 2.5.0.1

@frank42hh
Copy link
Author

frank42hh commented Jun 30, 2020

Moin, 2.5.0.1 ist nun installiert, aber irgendwie scheint da immer noch irgendwas falsch zu sein.
Es ist Dienstag, 10:20 Uhr. Ich habe die GEx noch nicht betreten, trotzdem liefert mir das Modul ein Ergebnis:

Anmerkung 2020-06-30 102223

Das spannende daran: Das kann unmöglich das Ergebnis von "seit Dienstag" sein, weil "seit heutigem Dienstag" müsste null sein, und "seit letzte Woche Dienstag" müsste deutlich mehr sein als ein Einlagerung-Kit.

@outoftheline outoftheline added Bug and removed Next Version Issue will be dealt with in the next update labels Jun 30, 2020
@outoftheline outoftheline reopened this Jun 30, 2020
@frank42hh
Copy link
Author

Noch zwei Screenshots vom heutigen Dienstag, nach 9 Begegnungen in der GEX, einmal "heute":
Anmerkung 2020-06-30 164629

und einmal "seit Dienstag"
Anmerkung 2020-06-30 164703

Morgen schiebe ich hinterher, wie es sich an einem Mittwoch darstellt mit diesen Einstellungen

@GeniusTimo
Copy link
Collaborator

Das Problem sind Lokalzeit, Serverzeit, ... Auf welchem Server spielst du und in welcher Zeitzone lebst du?

Repository owner deleted a comment from frank42hh Jun 30, 2020
@GeniusTimo
Copy link
Collaborator

Danke @dsiekiera 🙂 Wegen den Statistiken muss man mal schauen...

@frank42hh
Copy link
Author

de16, dementsprechend Europe/Berlin, also UTC+2.

Aber wie kann die Uhrzeit da relevant sein, wenn das Teil "x Tage zurück"
rechnet?

@GeniusTimo
Copy link
Collaborator

Muss man sich anschauen, braucht ohnehin eine Überarbeitung (GG startet Donnerstags)

@frank42hh
Copy link
Author

So, wie versprochen, heute nochmal ein paar Runden GEx gemacht, mit diesem Ergebnis:

Anzeige für "heute" ist korrekt:
Anmerkung 2020-07-01 133203

Die Anzeige "seit Dienstag" zeigt nur die heute gesammelten Belohnungen ein und ignoriert die von gestern.
Anmerkung 2020-07-01 133240

@Gindi4711 Gindi4711 added the Next Version Issue will be dealt with in the next update label Jul 1, 2020
@Gindi4711
Copy link
Collaborator

So habe mir das noch einmal angesehen:

moment().startOf('week') liefert je nach Regionaleinstellung entweder Sonntag 0 Uhr (amerikanischer Kontinent) oder Montag 0 Uhr (Europa/Asien)
moment().startOf('isoWeek') liefert immer Montag 0 Uhr

Ursprünglicher Code:
moment().startOf('week').add(2, 'days').toDate() liefert entweder Dienstag 0 Uhr (amerikanischer Kontinent) oder Mittwoch 0 Uhr (Europa/Asien)

Code in Version 2.5.0.1:
moment().startOf('isoWeek').add(2, 'days').toDate() liefert immer Mittwoch 0 Uhr (Montag 0 Uhr + 2 Tage weiter)

Neuer Fix:
moment().startOf('isoWeek').add(1, 'days').toDate() liefert immer Dienstag 0 Uhr (Montag 0 Uhr + 1 Tag weiter)

@GeniusTimo
Copy link
Collaborator

Ah, dann hatte ich das vertauscht - aber gut.. fast 😅

@frank42hh
Copy link
Author

Trotzdem Danke! :-)
Dann bin ich mal gespannt auf die nächste Version :)

@mainIine
Copy link
Owner

mainIine commented Jul 3, 2020

Das darfst du sein. Dort kommt nämlich auch direkt ein Datepicker mit und die Ansicht wurde überarbeitet.

@frank42hh
Copy link
Author

Datepicker? Wie genial! Ein Punkt, den ich eigentlich noch ansprechen wollte, war die Auswahl der vorgegebenen Zeiträume... weil gerade wenn man monatliche Statistiken erfassen will, wäre neben "Dieser Monat" nämlich ein "Voriger Monat" praktisch (damit man nicht immer am Monatsletzten alles notieren muss und einen Tag später die Statistik nicht mehr notierbar ist).
Aber ein Datepicker macht das natürlich deutlich angenehmer :)

@mainIine
Copy link
Owner

Der Datepicker kommt erst in der nächsten Version.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Next Version Issue will be dealt with in the next update
Projects
None yet
Development

No branches or pull requests

5 participants