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: use timestamp from multiplexed log #747

Merged
merged 1 commit into from
Mar 21, 2019

Conversation

tkurki
Copy link
Member

@tkurki tkurki commented Mar 17, 2019

Override N2K timestamp with the value from the multiplexed log.

Depends on a change in canboat.

@sbender9
Copy link
Member

canboatjs has been released with the required change.

@tkurki tkurki force-pushed the multiplexed-timestamp-override branch from 67cf173 to 90de7ce Compare March 20, 2019 20:51
Override N2K timestamp with the value from the multiplexed log.
@tkurki tkurki force-pushed the multiplexed-timestamp-override branch from 90de7ce to 57b6044 Compare March 20, 2019 20:55
@tkurki
Copy link
Member Author

tkurki commented Mar 20, 2019

Rebased & bumped canboat/canboatjs version.

@sbender9
Copy link
Member

Is this still WIP?

@tkurki
Copy link
Member Author

tkurki commented Mar 20, 2019

No.

@tkurki tkurki changed the title [WIP] feature: use timestamp from multiplexed log feature: use timestamp from multiplexed log Mar 21, 2019
@tkurki tkurki merged commit b5bec19 into master Mar 21, 2019
@tkurki tkurki deleted the multiplexed-timestamp-override branch March 21, 2019 16:06
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