set-in-obsidian-plugin icon indicating copy to clipboard operation
set-in-obsidian-plugin copied to clipboard

[BUG?] Recurring events not supporting time and duration arguments

Open NothingWorksToday opened this issue 2 years ago • 4 comments

Hey (again),

is this expected behavior or an already known issue?

Problem

When specifying recurring events with rrule-syntax, any time or duration arguments are ignored.

Example

Using

- `2023-01-16T14:30 PT2H every week on Monday` test Event

and a query like

set-in-obsidian-minimal
{
	"defaultView": "listDay",
    "fullcalendar": {
        "initialDate": "2023-01-16",
        "listDayFormat": false,
        "eventOrderStrict": true
     },
	"showGlobalEvents": "true",
	"files": ["_events"]
}

results in Bildschirm­foto 2023-01-23 um 21 26 57

Possible issue

Could it be that the field dtstartfrom FullCalender is not fully implemented yet? This field supports event time as per their documentation.

Solution

Please support the T and the PT arguments in recurring events.

NothingWorksToday avatar Jan 23 '23 20:01 NothingWorksToday

Duration in recurring events is not supported at the moment, i guess i missed that 😅

Ill see if i can add it but im busy at the moment

EDIT: also it seems it seems to ignore starting time of recurring events, ill have to fix that as well..

sandorex avatar Jan 24 '23 10:01 sandorex

I've made a discussion https://github.com/sandorex/set-in-obsidian-plugin/discussions/7 if you have any ideas for new format, im all ears

sandorex avatar Jan 24 '23 10:01 sandorex

As I know absolutely no TS, is it enough to look at the function parseListItem(raw: string) to help you and make suggestions? Or any other places to look after?

NothingWorksToday avatar Jan 24 '23 21:01 NothingWorksToday

All available formats are in the example in README so there is no need to look that mess of code 😅

sandorex avatar Jan 24 '23 22:01 sandorex