search

MaxMaeder / DriveBackupV2

Uploads Minecraft backups to Google Drive/OneDrive or by (S)FTP
https://dev.bukkit.org/projects/drivebackupv2
MIT License
160 stars 49 forks source link

support daylight saving time #178

Closed StillGreen-san closed 3 months ago

StillGreen-san commented 4 months ago

this changes the type of dateTimezone of the advanced config section to ZoneId to support daylight saving time

currently you need to update date-timezone manually to account for DST. with this, specifying a zone id will take care of it automatically

very few changes where necessary as the previous ZoneOffset extends ZoneId and the functions used with dateTimezone already took ZoneId as arguments ZoneId.of parses both offsets and ids, so you can now specifiy either an offset or a zone in the date-timezone config field

the java docs reference the IANA Time Zone Database, but there is a nicer list from Noda Time a bunch of additional formats are also supported but the wiki entry should probably only mention ±HH:MM and {Area}/{City} for simplicity

rhld16 commented 3 months ago

Updating the wiki entry now to mentioned recommended formats + link the NodaTime list