search

tomhollander / PowerwallCompanion

Windows UWP app for monitoring your Tesla Powerwall 2 battery
9 stars 7 forks source link

Export appends to target file instead of overwriting #14

Closed peppersass closed 7 months ago

peppersass commented 7 months ago

Title says it all.

Previous versions overwrote the target file. It's debatable whether the app should append or overwrite, but in my workflow overwrite is best. For example, sometimes I need to replace faulty Day files created by the Tesla app and would rather not have to delete the file before the export or remove the old data after the export. In another case, I've added a feature to my data analysis app that relies on the day-by-day information in the Month export file. Each day, I download both the Day file from the previous day and the updated Month file. I have to delete the old Month file to avoid the append.

If append is a feature, not a bug, I'd rather not see a prompt asking whether I want to append or overwrite. Perhaps a Setting for append or overwrite is the way to go.

tomhollander commented 7 months ago

Thanks, this wasn't deliberate. I'll look into this when I get some time.

tomhollander commented 7 months ago

Fixed in 5.3.2.