search

viktorgullmark / exilence-next

Desktop application that helps you summarize your wealth in Path of Exile
648 stars 125 forks source link

Remove Snapshot datetime format is incorrect #861

Open HeartofPhos opened 1 year ago

HeartofPhos commented 1 year ago

Describe the bug The format provided to moment.js in ExilenceNextApp\src\components\remove-snapshots-dialog\RemoveSnapshotsDialog.tsx:100 is incorrect, the given format is 'YYYY-MM-DD HH:MM' instead of 'YYYY-MM-DD HH:mm' resulting in the month of the snapshot being rendered in the minutes positions

To Reproduce Steps to reproduce the behavior:

  1. Take at least 1 snapshot
  2. Open then Remove Snapshots dialogue
  3. Observe the minutes value is equal to the current month and not the minutes of the snapshot
C3ntraX commented 1 year ago

This will be fixed in my PR #863