-
Notifications
You must be signed in to change notification settings - Fork 19
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
Fixes #400 #401
Fixes #400 #401
Conversation
I have not updated the documentation on the website, as I believe it is automatically generated from the schema itself. |
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.
I am but a bit player here, but I like it!
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.
@RS-Credentive Thanks for the submission. The XML changes look good.
Can you make the same changes to the JSON data types? These need to be kept in sync.
No problem. Thanks for the heads up. I thought they were autogenerated from the metaschema, or I would have already done so. |
Updated JSON to reflect the new timezones regex you recommended. |
Corrected TZ pattern Co-authored-by: David Waltermire <[email protected]>
Corrected regex for TZ in Date Co-authored-by: David Waltermire <[email protected]>
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.
Looks good. Thank you for the additional edits.
Committer Notes
Addresses
#400
All Submissions:
Changes to Core Features: