cesium / calendarium

📆 Calendar with special events, due dates and week schedule
https://calendario.cesium.di.uminho.pt/
11 stars 2 forks source link

fix: bug in event fetching for `groupId = 0` #165

Closed diogogmatos closed 11 months ago

diogogmatos commented 11 months ago

Fixed a bug that caused events with the groupId parameter set to 0 to be considered invalid. This happened because the value 0 would be read as false in the if condition.

In the process of solving this, I also made some improvements to the way dates and times are shown in EventModal, particularly for events that extend for multiple days AND have a specific start hour and end hour, for example Karaoke, that goes from 28/11 at 21:00 to 29/11 at 02:00.

Also made the event location a link that takes the the user to Google Maps and automatically searches for the location.

netlify[bot] commented 11 months ago

Deploy Preview for cesium-calendarium ready!

Name Link
Latest commit 3fae33e093c68268d2e393d9daf0631748621af3
Latest deploy log https://app.netlify.com/sites/cesium-calendarium/deploys/65608c25d886ca0008716912
Deploy Preview https://deploy-preview-165--cesium-calendarium.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.

diogogmatos commented 11 months ago

fix: bug in event fetching for groupId = 0