Problem/Motivation

This functionality would be really useful on the smart date module https://www.drupal.org/project/smart_date but it currently doesn't work because Smart Date uses its own custom entity. This patch makes that integration work

CommentFileSizeAuthor
1551026-smart_date.patch1.31 KBrloos289

Comments

rloos289 created an issue. See original summary.

stijndmd’s picture

stijndmd commented

This patch works.

However, it doesn't catch recurring smart dates, it just adds the final date from the recurring to the calendar.

rloos289’s picture

rloos289 commented

Correct, my original requirement was just to get this to work for singular events. I haven't yet looked into getting recurring dates to work, but if there is interest in that functionality then that could be a good next step for this patch.

jdearie’s picture

jdearie commented

Thank you - this patch worked for us.
+1 for support for recurring dates. Until then, we have to add some language letting users know that only the first instance of the event will be added to their calendar.

smustgrave’s picture

smustgrave commented

I had a lot of luck with https://www.drupal.org/project/smart_date_calendar_kit which used fullcalendar.

matt_paz’s picture

matt_paz commented

The patch works well enough for recurring dates with "Allow recurring date values" set for the range. It shows each instance with that setting, but it would be even better if we could find a way to render the date itself with the smart date formatter.

mandclu’s picture

mandclu commented

FYI there is now support for recurring dates in add to calendar links in Add to Calendar Date Augmenter: https://www.drupal.org/project/addtocal_augment Any feedback would be appreciated.

matt_paz’s picture

matt_paz commented

@mandclu - I haven't gotten around to testing it yet, but both it (and date_augmenter) look interesting. Thanks for sharing!

coaston’s picture

coaston commented

Just tested and the patch works. Thank you!

  • wxactly committed 08e9149 on 8.x-2.x authored by rloos289 
    Issue #3200494 by rloos289: Add smart date support
wxactly’s picture

wxactly commented
Status: Needs review » Reviewed & tested by the community

Thanks all - this will be in the next release of the module. If anyone makes more progress on supporting recurring smart dates, please open a new issue with a patch.

wxactly’s picture

wxactly commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.