-
Notifications
You must be signed in to change notification settings - Fork 33
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
ISO8601 is more than just YYYY-MM-DD #25
Comments
Though they are valid under ISO8601, by my reading they are not valid under XML schema definition of dateTime, which is why we chose the name xDate, to make it clear we were following that definition (and because our use case required a full date), so while I take your point that a wider range of date format options may well be appropriate, I think they need to be new options rather than changing the definition of xDate. From our point of view as an archive the features of the Extended Date/Time Format which are apparently expected to be included in ISO8601 revision this year are likely to be even more useful as very often our dating is necessarily imprecise. |
Ok, that’s fair. Although the spec says
Sure, I’m fine with any option. ☺ For what it’s worth, in my current use case I need a way to specify that the column can be any of YYYY, YYYY-MM, or YYYY-MM-DD, each row potentially following any of these. I ended up specifying it as |
It's there mostly because XML Schema 2 also refers to it, and people are broadly familiar is ISO8601 at least meaning year first, but there might be a slightly better way of expressing that. I think I'd test with Depending where you're getting your data from we've often found it better to specify separate columns for the constituent parts of a date where there may be a mixture of formats, and that also makes it easier to handle invalid dates, such as 31 April or 29 February in a non-leap year. Clerks of the past wrote down impossible dates surprisingly often which is pain when you're handling transcribed data. |
Actually even better might be eg |
YYYY and YYYY-MM should also be accepted as valid for xDate fields.
The text was updated successfully, but these errors were encountered: