Post event date not always displayed on topic title

Bug
1

I have display post event date on topic title set to true, but some event don’t get their event date thing displayed near the topic title.

Most of them have, though, the category is here:

In which:

  • Pour les grenoblois - Meetup le 28 mars have its event date near topic title: dans 8 jours
  • Soirée des communautés techniques rennaises don’t have its event date near topic title and I don’t know why.
  • Pour les lyonnais − Meetup le 22 mars have its event date.

Interestingly the one with no tag still appear in /upcoming-events.

2 Likes
2

Hi Julien,

I’ve checked a little, and I’m wondering if it’s because you don’t provide an end date for that event. :thinking:

5 Likes
3

Hummm I can test that!

Yes that was it. Feel strange though…

2 Likes
5

I understand what you mean. That would be welcome to have the end date optional. I’ve checked the code, and the logic could be improved here to allow this. :slight_smile:

1 Like
6

+1 from me if this could be improved. It’d really help my community.

IRL events are scheduled, often with no end time. When there’s a list of events shown in the category, it’s pretty confusing when only a few are shown with the start time after the topic title.

Additionally, if an end time is defined, the event is removed from the upcoming calendar if it has already occurred so the incentive for my users is to not define an end time. Like I said, it’s a bit confusing.

Happy to help out if I can.

1 Like
7

I also hit this. Here’s a proposed fix: Show topic title suffix even when missing an ending by mbauman · Pull Request #501 · discourse/discourse-calendar · GitHub

1 Like
8

Any feedback on the above proposed change?

9

Looks like there are no tests, any chance you can add a test here?

10

It would be great to finish this off - have been scratching my head as to why some events seemed to randomly be not showing the date / countdown in the topic lists.

Now I understand it, and have a workaround by simply adding an end date. The suggested fix looks pretty good to my very amateur eye (lack of tests not withstanding).

14

This will be fixed once this PR gets merged. Thanks @mbauman for the inspiration :+1:

5 Likes
16

This topic was automatically closed after 3 days. New replies are no longer allowed.