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.

Sign up for GitHub

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

Jump to bottom

Refine "expectedDuration" property in schema #216

Closed
bquinn opened this issue Nov 11, 2024 · 0 comments · Fixed by #218 or #157
Closed

Refine "expectedDuration" property in schema #216

bquinn opened this issue Nov 11, 2024 · 0 comments · Fixed by #218 or #157
Assignees
@bquinn
Milestone
3.0

Comments

@bquinn
Copy link
Contributor

bquinn commented Nov 11, 2024

We agreed to refine the "duration" property to refer to iCalendar durations: https://icalendar.org/iCalendar-RFC-5545/3-3-6-duration.html

We said that we would just refer to "iCalendar duration" in the schema description comment, and in the documentation we would refer to the above website.

We should also include an example / test case showing how it would be used.
@bquinn bquinn added this to the 3.0 milestone Nov 11, 2024
@bquinn bquinn self-assigned this Nov 11, 2024
@iyoung iyoung changed the title Refine "duration" property in schema Refine "expectedDuration" property in schema Nov 11, 2024
bquinn added a commit that referenced this issue Nov 11, 2024
@bquinn
Added refinement to specify expectedDuration property. Fixes #216.
8cae52e
@bquinn bquinn linked a pull request Nov 11, 2024 that will close this issue
Added refinement to specify expectedDuration property. Fixes #216. #218
Merged
bquinn added a commit that referenced this issue Nov 13, 2024
@bquinn
Merge pull request #218 from iptc/fix/expectedDuration-desc
e224a44 
Added refinement to specify expectedDuration property. Fixes #216.
@bquinn bquinn linked a pull request Nov 14, 2024 that will close this issue
ninjs 3.0 release candidate, including events, planning, live streams and camelCase properties #157
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@bquinn bquinn

Labels
None yet
Projects
None yet
Milestone
3.0
1 participant
@bquinn