-
Notifications
You must be signed in to change notification settings - Fork 3.7k
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 examples for duration and period granularities #1832
Conversation
f8a3954
to
538696a
Compare
Thanks for the examples. While you're in this file, perhaps you could add a period at the end of the sentence "Supported granularity strings are: ..." Also, it would be nice (and more consistent) to provide an example under "Simple Granularities" like the ones for the other granularity types. That might help show why the specification is described as being given by a string or an object - I was initially confused about what "object" means in this context. |
👍 |
538696a
to
930ecb5
Compare
+1 for adding examples to the other "simple" granularity types. Alternatively, you can just indicate what the string types are equivalent to (except none and all for which it doesn't really make sense to give the object form) |
As a general note in case some DEV stumbles here later, Found that one the hard way the other day. |
930ecb5
to
26e0b58
Compare
Done adding the period and adding examples to "simple" granularity. |
26e0b58
to
6a30e19
Compare
Thanks. LGTM. |
👍 |
} | ||
], | ||
"intervals":[ | ||
"2000-01-01T00:00Z/3000-01-01T00:00Z" |
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.
can you change the interval also to America/Los_Angeles timezone? that would be the common use case.
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.
Done.
6a30e19
to
4030908
Compare
4030908
to
2de5e9c
Compare
add examples for duration and period granularities
add examples for duration and period granularities