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

Add absent RRuleSet getters (GetRDate(), ...), extract StrSliceToRRuleSet() and StrToDates() functions #10

Merged
merged 5 commits into from
May 18, 2018

Conversation

egorbunov
Copy link
Contributor

@egorbunov egorbunov commented May 17, 2018

  1. GetRDate() and other getters for rrule Set seem to be useful when working with rrule Set after parsing it from string
  2. StrSliceToRRuleSet() takes array of strings, each encoding one RRULE, EXDATE, and so on. This one may be useful when not single string encoding rrule Set is passed to application, but array of strings as it is done when using Google Calendar API
  3. StrToDates() added and made exported so that user may implement it's own logic during rrule Set construction and decide which parsed from string EXDATE/RDATE should be included in the set and which should not
  4. Also I've made date-time format string exported too so that package user may rely on it somehow
  5. Also simple test were added on parsing rrule set from string

Egor Gorbunov added 5 commits May 17, 2018 16:13
This one is useful  in case your app has to deal with array of strings
encoding recurrence set from the beginning, for example if you receive
event json response from google calendar api, recurrence set is encoded
with an array of strs there, not with single str with all rules concated
@coveralls
Copy link

Coverage Status

Coverage increased (+3.7%) to 98.297% when pulling a9069c8 on egorbunov:master into a8479ea on teambition:master.

@egorbunov egorbunov changed the title Add absent RRuleSet getters (GetRDate(), ...), extract StrSliceToRRuleSet() and StrToDates() functions Add absent RRuleSet getters (GetRDate(), ...), extract StrSliceToRRuleSet() and StrToDates() functions May 17, 2018
@egorbunov egorbunov changed the title Add absent RRuleSet getters (GetRDate(), ...), extract StrSliceToRRuleSet() and StrToDates() functions Add absent RRuleSet getters (GetRDate(), ...), extract StrSliceToRRuleSet() and StrToDates() functions May 17, 2018
@zensh zensh self-assigned this May 18, 2018
@zensh zensh requested a review from damoye May 18, 2018 02:40
@zensh zensh merged commit fd30717 into teambition:master May 18, 2018
@egorbunov
Copy link
Contributor Author

egorbunov commented May 21, 2018

Thanks 😎 But review is still expected, I guess :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants