-
-
Notifications
You must be signed in to change notification settings - Fork 20
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add RECURRENCE-ID with range parameter. #181
Conversation
d4a71f1
to
b09621c
Compare
b09621c
to
bb8dc97
Compare
WIP: Need to include rrule |
Added test for rdate |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi, I request changes but first of all! Cool! This looks really good! Let me know which changes you would like to tackle and which ones not so I can create a PR to your branch for those that you do not like ;)
Also, to close the issue we need more tests, I believe..
Ready for review again, thanks for get into! |
Thanks for your additions! I will merge this and open another PR with the additional tests.. This does not yet solve the issue but it is really close. |
Add RECURRENCE-ID with range parameter.
contribute to #75