Expired events with end times are being removed from upcoming-events even with setting set to -1

Helen22 · July 27, 2022, 8:45am

I am trying to utilise the Calendar Plugin, but it deletes the event from the calendar after it happens despite the fact that the settings are set to -1 to disable the deletion. I’ve also tried setting the expiry to two years (17532hrs) and that also removed the event straight after it happened.

The only way I’ve found to keep the events in the calendar is to remove the end time/date.

Does anyone know why the -1 doesn’t work, and what the solution is?

Thanks

JammyDodger · July 27, 2022, 9:11am

Hello and welcome @Helen22

I’ve been reviewing this plugin recently and I handily have a couple of calendars on my test site to check against. Could I ask which flavour of calendar you’ve gone for where this isn’t working?

I have the calendar in the topic, and that one is keeping the expired calendar items:

I also have it as a category header:

And I also have an /upcoming-event one too for the event-style items:

I haven’t got an event-style one as a category header yet, so I have a feeling it may be that one…

Actually, let me double-check the end times…

Yes, it appears adding an end time will make the event disappear from the /upcoming-event calendar, which does seem to be at odds with the other calendar behaviour. I think we should slide this over to the #bug category and see if we can get someone to look at it.

(and, just to be thorough, the non-event-style calendar is working as expected, even with end dates included)

Benjamin_D · July 27, 2022, 10:01am

If you’re referring to the setting delete expired event posts after, I think it prevents the deletion of the post, but not the display of the event, plus I think the term event here is misleading, it’s in the calendar section so I think it means more “date related thing” than “event in the rsvp-possible sense”.
that’s a calendar-event, not an event-event

event-events are meant to be displayed in /upcoming-events hence nothing in the past…

github.com

discourse/discourse-calendar/blob/e544ef61fd90b25a2e984c30cb4efa00ee8051f2/lib/discourse_post_event/event_finder.rb#L17-L25


      
          
          
if params[:expired]
            # The second part below makes the query ignore events that have non-expired event-dates
            events = events
              .where("dcped.finished_at IS NOT NULL AND (dcped.ends_at IS NOT NULL AND dcped.ends_at < ?)", Time.now)
              .where("discourse_post_event_events.id NOT IN (SELECT DISTINCT event_id FROM discourse_calendar_post_event_dates WHERE event_id = discourse_post_event_events.id AND finished_at IS NULL)")
          else
            events = events.where("dcped.finished_at IS NULL AND (dcped.ends_at IS NULL OR dcped.ends_at > ?)", Time.now)
          end

There is a PR about that actually: FEATURE: Show all events in past (add deleted_at in discourse_calendar_post_event_dates) + add show_past_event in settings by ledadu · Pull Request #199 · discourse/discourse-calendar · GitHub

JammyDodger · July 27, 2022, 10:08am

I couldn’t decide if the quirk was that it did show past events as long as they didn’t have an end time, or that it didn’t show the ones with an end time. It perhaps may be something I could include in the plugin topic rewrite if it’s a case of expectation versus actual behaviour?

Benjamin_D · July 27, 2022, 10:21am

could an event be in the past if it doesn’t have an end time?

Car Chase Action GIF by Regal

I think the behavior now is expected, not showing past ones (ie with an end time in the past), but an option to be able to show all of them could be nice (but have unexpected UI side effect, I’m thinking here for instance: GitHub - paviliondev/layouts-event-list-widget: The Event List Widget allows you to display upcoming events in a sidebar using Pavilion's Custom Layouts Plugin.)
If you’re going for a rewrite of the topic, pleeease “event” should really be exclusively used to describe events related things, it’s very confusing when related to calendar

JammyDodger · July 27, 2022, 10:32am

Yup. And possibly even a tweak to some of the setting descriptions as well.

Rather than ping pong it to another category just now, I’ll leave it here and see if I can absorb some further clarity before finding it a forever home.

But thank you for being so knowledgable about this plugin, it is a genuine help.

Helen22 · July 28, 2022, 6:05am

Thanks all for your comments -

We have the upcoming-events calendar from the hamburger menu, and the upcoming-events in the category. Neither of them are retaining the events.

Where do I find this?

I agree with this, as visually the upcoming events calendar looks more utilised when it is populated vs when there’s nothing to show in it.

JammyDodger · July 28, 2022, 9:07am

You can create a regular calendar by adding the following to the OP of a topic: (with calendar enabled checked)

[calendar]
[/calendar]

And then all of the following posts with a description and date will get added to it. eg:

It's my birthday!
[date-range from=2022-12-20 to=2020-12-21 timezone="America/Boise"]

You can then grab a category id of where you want to display it more prominently from the end of the category URL, and the post id of the calendar topic (you can get this from the json) and add it to this setting:

The calendar item posts in these topics are the ones controlled by the delete expired event posts after setting.

There’s more detailed info in the Discourse Calendar topic, ~~though if you need extra help feel free to spin up a new #support topic too.~~

Actually, this seems like a good juncture to slide this topic over to #support for its forever home now we know that the setting is working as expected. After I’ve updated the Calendar&Event literature it may be worthwhile creating new #ux or #feature topics to cover any missing ideas.

Helen22 · July 29, 2022, 6:55am

Thanks for your help @JammyDodger. I really appreciate it.

So just to circle back and clarify the answer my initial question, there is currently no way of keeping events in the upcoming-events calendar after they’ve happened, unless I remove the end time?

JammyDodger · July 29, 2022, 7:52am

No worries. Glad to help.

But yes, the only way to retain event-style items in the event-calendar/upcoming-events currently is to remove their end date. I wondered if another workaround to achieve something similar could be to create a ‘Past Events’ or ‘Event History’ calendar topic using calendar items? I’m not sure if that would be suitable for you, but may be worth an explore?

Helen22 · July 29, 2022, 8:24am

Yes, that might work. I will look into it. Thanks!

Topic		Replies	Views
Past events keep getting deleted support calendar-and-event	3	304	November 10, 2023
Category Calendar with defaultView="listNextYear" not showing events support calendar-and-event	22	1232	July 22, 2022
Display old Events on the 'Upcoming Events' page feature calendar-and-event , completed	9	908	January 9, 2024
Showing discourse events at the top of a category feature calendar-and-event	11	1006	April 10, 2021
Discourse Calendar (and Event) plugin official , calendar-and-event	19	43325	April 17, 2024

Expired events with end times are being removed from upcoming-events even with setting set to -1

Related Topics