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

feature: update tv4-formats to latest version #501

Merged
merged 2 commits into from
Sep 2, 2018

Conversation

sarfata
Copy link
Contributor

@sarfata sarfata commented Sep 1, 2018

Upgrade tv4-formats to latest version which works well with Webpack.
Previous versions used a try/catch import heuristic which does not play
well with webpack.

See ikr/tv4-formats@61450dc

(This PR replaces #498 - It also includes added tests so that my mistake in the first PR does not happen again).

Add two tests to make sure that timestamp validation is working.

The first test 'ending with Z' will verify that a timestamp not ending
with Z is rejected. This is the signalk spec regexp '.*Z'.

The second test is more subtle and requires the date to be an actually
valid timetamp. We currently use tv4-formats for this.
Upgrade tv4-formats to latest version which works well with Webpack.
Previous versions used a try/catch import heuristic which does not play
well with webpack.

See ikr/tv4-formats@61450dc
@tkurki tkurki changed the title feature: update tv4 to latest version feature: update tv4-formats to latest version Sep 2, 2018
@tkurki tkurki merged commit 106f31b into SignalK:master Sep 2, 2018
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