Skip to content
This repository has been archived by the owner on Sep 20, 2023. It is now read-only.

Ganache Mining Interval #985

Merged
merged 2 commits into from
Nov 23, 2018
Merged

Ganache Mining Interval #985

merged 2 commits into from
Nov 23, 2018

Conversation

micahalcorn
Copy link
Member

@micahalcorn micahalcorn commented Nov 23, 2018

  1. In April, blocktime was renamed to blockTime.
  2. I can't find a good reason why using the blockTime flag is discouraged.
  3. With an undefined value for blockTime, I don't know how our blockchain transaction confirmation UI ever worked with Ganache. Regardless, it currently doesn't in my experience, and this fixes it.

@sparrowDom
Copy link
Member

NIce @micahalcorn ! I have noticed on Ganache that block confirmations in transaction dropdown increase only when a new transaction is mined.

@micahalcorn micahalcorn removed the request for review from tomlinton November 23, 2018 16:46
@micahalcorn micahalcorn merged commit a6f6b86 into master Nov 23, 2018
@micahalcorn micahalcorn deleted the micah/mining-interval branch November 23, 2018 17:07
micahalcorn added a commit that referenced this pull request Nov 23, 2018
tomlinton pushed a commit that referenced this pull request Nov 23, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants