beastsaber / bsaber

https://bsaber.com
9 stars 9 forks source link

Fix how dates are displayed in the events section in regard to local timezones #107

Closed Jonas00000 closed 5 months ago

Jonas00000 commented 5 months ago

Fix of issue #86

netlify[bot] commented 5 months ago

Deploy Preview for bsaber ready!

Built without sensitive environment variables

Name Link
Latest commit fb12303c99b515444df50fb5d732f35908bfe198
Latest deploy log https://app.netlify.com/sites/bsaber/deploys/661aba67224aa900084e1e60
Deploy Preview https://deploy-preview-107--bsaber.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

TimothyGillespie commented 5 months ago

Looks good to me. I checked it out locally and verified it. I did notice, when a endTime is given, but not endDate the endtime gets ignored. That seems to make sense to me though and has been the previous behavior as well.

I don't see an error as of now here.

@Pixelguymm you may want to check it out, since it's also a partial self-review being that I gave some advised/opinion on this during the process.