Strip date of milliseconds to fix 4:07 error #301
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Overview
The date time we get from TCAT has changed format sometime over the summer. They now include milliseconds unfortunately. We need to revert this format back to what it was so that clients don't break.
Date format we had before that we should maintain:
yyyy-MM-dd'T'HH:mm:ssZZZZ
Date format that TCAT changed to over the summer:
yyyy-MM-dd'T'HH:mm:ss.SSSZ
Changes Made
docker-compose.yml
so we can run things locally with this changeghopper
Dockerfile
because Tom sent me data manually (can't fetch from the URL) so to keep versions matching with docker images, we should have master keep this change--once we can start fetching again (Aug 30), I'll update masterTest Coverage
transit-node:v1.1.4
transit-dev:date
Screenshots
Error on the left, fix on the right