rrule icon indicating copy to clipboard operation
rrule copied to clipboard

Adding the dtstart property in the RRULE

Open ericyunwen opened this issue 1 year ago • 3 comments

If you look at a more elaborated approach here http://jkbrzt.github.io/rrule/,

You will notice the property dtstart which is the start date of the rule. It should work similarly but opposite to the until property.

It will be great to have this included as a property. I also will like to work on this if no one is available to contribute

ericyunwen avatar Jan 11 '24 10:01 ericyunwen

DTSTART is not part of the RRULE, it's a seperate property of an iCalendar object. So I am not sure, if this would be the best way to handle the property, if there is start as parameter...IMO this is not needed. Do you have a specific use case?

In the end @JonasWanke has to decide if this should be implemented as rrule.js does it. This package is based on rrule.js already.

jonasbadstuebner avatar Jan 23 '24 17:01 jonasbadstuebner

I lean towards not including it inside the RecurrenceRule for the reason mentioned by DrBu7cher above. We could maybe add a wrapper class that can store start and the recurrence rule (possibly being generic so it also supports RecurrenceSets (#1) in the future), but I don't really think this class would be worth it.

JonasWanke avatar Jan 24 '24 19:01 JonasWanke

I also would like this, I mistakenly thought dtstart was part of rrule's in general and introduced a bug because of that, as I was doing everything server side before, and old rrules which had dtstart don't parse. I do think it's a useful feature to have as it does have revelance to how rrules should schedule items.

hamishjohnson avatar Jan 28 '24 23:01 hamishjohnson