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

[.NET Timexlib] TimexProperty.ToString() does not support daterange/datetimerange and other complex variants #2904

Open
shana opened this issue Mar 15, 2022 · 0 comments · May be fixed by #2903

Comments

@shana
Copy link
Contributor

shana commented Mar 15, 2022

Describe the bug
Following up on #2894, TimexProperty.ToString() is incomplete when it comes to supporting string representations of all of the different timex types.

Examples that it doesn't support:

Timex Should probably output
(2022-03-15T16,2022-03-15T18,PT2H) 15th March 2022 4PM to 6PM
(2022-03-15T16,2022-03-17T16,PT50H) 15th March 2022 4PM to 17th March 2022 6PM
(2022-SU,2022-SU,PT3H40M) summer 2022 3 hours 40 minutes
(2022-W02-6,2022-W03-6,P7D) Saturday week 2 2022 to Saturday week 3 2022
(2022-W02-WE,2022-W03-WE,P7D) weekend of week 2 2022 to weekend of week 3 2022

There's probably others that I'm not thinking of at the moment, but this encapsulates a lot of missing functionality. PR #2903 is a stab at implementing these and fixing #2893 properly.

Platform (please complete the following information):

  • Platform: .NET
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 a pull request may close this issue.

1 participant