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

Working on fixing ISO8601 parsing #15

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

ciozi137
Copy link
Contributor

@francois-normandin this version is parsing more variants but is probably still incomplete

@ciozi137
Copy link
Contributor Author

@francois-normandin Change summary:

ISO8601 DateString to Timestamp.vi: UTC Date-Time Case:

image

ISO8601 DateString to Timestamp.vi: ISO Time Case:

image

Timestamp to ISO8601 Time.vi

  • a new method called by DateTime.ISO8601.lvclass:toString.vi

image

Test Files

image

Test ISO8601 DateTime Strings.vi:

image

@francois-normandin
Copy link
Member

Thanks @ciozi137 !
I'll probably review this in a couple weeks, so don't be alarmed by my silence in the coming days...

@ciozi137
Copy link
Contributor Author

hey @francois-normandin no problem - understood!

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 this pull request may close these issues.

2 participants