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.

Sign up for GitHub

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

Jump to bottom

Update Confirmation Timers to be ms not sec #1442

Merged
merged 1 commit into from
Dec 12, 2018
Merged

Update Confirmation Timers to be ms not sec #1442

merged 1 commit into from
Dec 12, 2018

Conversation

argakiig
Copy link
Contributor

@argakiig argakiig commented Dec 12, 2018 •
edited
Loading

Confirmation History avg and per block timers to be in MS as the rest of the timers are
declaration more easily readable

@rkeene rkeene added this to the V17.0 milestone Dec 12, 2018
@rkeene rkeene requested a review from cryptocode December 12, 2018 06:03
@rkeene rkeene added the quality improvements This item indicates the need for or supplies changes that improve maintainability label Dec 12, 2018
@rkeene rkeene merged commit 8d8a4b6 into nanocurrency:master Dec 12, 2018
rkeene pushed a commit that referenced this pull request Dec 12, 2018
@argakiig @rkeene
Update Confirmation Timers to be ms not sec (#1442)
0fdc2c0
@argakiig argakiig deleted the confirmation_timer_to_ms branch December 15, 2018 05:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cryptocode cryptocode cryptocode approved these changes

Assignees

@argakiig argakiig

Labels
quality improvements This item indicates the need for or supplies changes that improve maintainability
Projects
None yet
Milestone
V17.0
Development

Successfully merging this pull request may close these issues.
3 participants
@argakiig @cryptocode @rkeene