Set up Smart Date to work with FullCalendar View. Not clear what actual changes are needed, so creating this as a stub. If anyone knows what specific steps are required, please add to this description.

Comments

mandclu created an issue. See original summary.

MrPaulDriver’s picture

MrPaulDriver CreditAttribution: MrPaulDriver commented

You might also want to consider Full Calendar View. I get the impression there is more activity there.

This issue is telling: https://www.drupal.org/project/fullcalendar/issues/2920904

michaelschutz’s picture

michaelschutz CreditAttribution: michaelschutz commented

I'd echo the focus on Full Calendar View over Full Calendar. I'm using Full Calendar View on a project, and have it working fine with core Date Range fields. It's got significant quirks, but is a great module IMO.

When I learned about Smart Date, I thought it would be a great improvement over core Date Range for my project, so I tried it out. But Full Calendar View wouldn't display any data from a Smart Date field.

I'm not a developer, but I think one (if not the) issue is that Smart Date stores data as timestamps, and core uses datetime. If Full Calendar View is only able to parse datetime fields, then that would explain it. I don't know if this is a trivial thing to implement or a challenging one, but if it is the issue, it seems like the onus would be on Full Calendar View to work with Smart Date, not the other way around.

mandclu’s picture

mandclu CreditAttribution: mandclu at Northern Commerce commented
Title: Integration with FullCalendar » Integration with FullCalendar View
Issue summary: View changes
Related issues: +#3061341: provide support for Smart Date

Thanks for the suggestion, I've updated this issue to reflect that.

mandclu’s picture

mandclu CreditAttribution: mandclu at Northern Commerce commented
Status: Active » Needs review

There's now a patch up at #3061341: provide support for Smart Date. Please test and provide feedback there.

mandclu’s picture

mandclu CreditAttribution: mandclu at Northern Commerce commented
Status: Needs review » Reviewed & tested by the community

Moving this to RTBC, to reflect the status of the Fullcalendar View issue. Potentially this could be closed outright, since it doesn't appear that any work is required within Smart Date for the integration to work.

mandclu’s picture

mandclu CreditAttribution: mandclu at Northern Commerce commented
Status: Reviewed & tested by the community » Closed (duplicate)

Closing this in favor of #3061341: provide support for Smart Date.