Skip to content
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

Support BYSETPOS for MONTHLY AND YEARLY freq #349

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

NicolasMarlier
Copy link

Self-explanatory ;)

@avit
Copy link
Collaborator

avit commented Mar 20, 2017

Interesting! This option doesn't seem to be clearly documented in the icalendar RFC but it looks like it's widely supported by different tools. I believe it would be worth finishing this up and adding it.

http://www.kanzaki.com/docs/ical/recur.html

@avit avit added the feature label Mar 31, 2017
@seejohnrun seejohnrun self-requested a review April 30, 2018 13:55
nehresma added a commit to nehresma/ice_cube that referenced this pull request Jul 30, 2018
A few small updates to Nicolas Marlier's BYSETPOS support added in
PR ice-cube-ruby#349
joshbeckman added a commit to officeluv/ice_cube that referenced this pull request Feb 5, 2019
In August 2016, @NicolasMarlier added BYSETPOS support (ice-cube-ruby#349).

Then, in July 2018, @nehresma added a few small changes to run in modern
Ruby and a more modern rspec
(ice-cube-ruby#449).

Then, in January 2019, @davidstosik and @k3rni suggested changes to
reduce complexity.

This incorporates all the above into a single diff.
davidstosik pushed a commit to davidstosik/ice_cube that referenced this pull request Feb 6, 2019
A few small updates to Nicolas Marlier's BYSETPOS support added in
PR ice-cube-ruby#349
nehresma added a commit to nehresma/ice_cube that referenced this pull request Dec 12, 2022
A few small updates to Nicolas Marlier's BYSETPOS support added in
PR ice-cube-ruby#349
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants