Problem/Motivation

The Smart Date module stores values as timestamps, so would need a little work for those values to display in a Calendar view.

Proposed resolution

Issues like #2820803: Support Recurring Dates Field in Calendar 8.x-1.x patch Calendar to work with other contrib modules. Could Calendar create a processor or some other kind of plugin class that other modules could implement in order to define their own integration? For example, Fullcalendar View has the FullcalendarViewProcessorBase that modules can extend to ensure the data is passed in with the correct data structure.

API changes

Provide some method for contrib modules to define their own compatibility with Calendar.

Note that I have also created #3177760: Support Calendar module specifically to address the work for Smart Date to implement whatever integration method is available.

Issue fork calendar-3177761

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

mandclu created an issue. See original summary.

mandclu’s picture

Issue summary: View changes
mandclu’s picture

Issue summary: View changes
derekb’s picture

Following - I had to switch to FullCalendar view because it supported SmartDate but Calendar was a lot easier to work with and theme.

sseto’s picture

I'm using SmartDate with FullCalendar view and it's really destroying my server because of loading all the historical events at once. I really hope SmartDate will work with Calendar.

grahamc’s picture

The required changes for Smart Date are actually way smaller than the patch targeting date_recur...

I have it up and running with the attached one-line diff, in conjunction with Smart Date patch to add granularity arguments.

(Yes, calling out specific module names is icky! - either an interface or otherwise inferring correctly from the field definitions would still be preferable)

mandclu’s picture

Added some comments to #3177760: Support Calendar module, not sure if any of them really apply more to this issue.

apkwilson’s picture

Status: Active » Needs review

I've just set this up using #6 and the patch in #3177760-13: Support Calendar module. Aside from having to add the date contextual filter before selecting the calendar format, the experience was seamless.
Agree that referencing smartdate isn't the best, but what would be required to avoid it? A new plugin type?

daniel korte’s picture

Status: Needs review » Closed (outdated)

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.