saade / filament-fullcalendar

The Most Popular JavaScript Calendar as a Filament Widget
MIT License
282 stars 85 forks source link

Error if the event ID is a UUID #80

Closed billmn closed 1 year ago

billmn commented 1 year ago

Using a UUID, instead of an auto-incremental ID, results in the following error when clicking on an event:

Cannot assign string to property Saade\FilamentFullCalendar\Widgets\FullCalendarWidget::$event_id of type ?int

This because in CanManageEvents trait, the $event_id property is typed as an integer. I think it can be typed as a string without causing any BC.

saade commented 1 year ago

sorry for the delay, its merged