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

Reopen ganache to restart development network, the MeatMask transcation state isn't refreshed right. #3629

Closed
3for opened this issue Mar 20, 2018 · 2 comments

Comments

@3for
Copy link

3for commented Mar 20, 2018

I'm now following the website below to try to use truffle and ganache in windows 10 with Chrome browser 64.0.3282.186 and MetaMask 4.2.0:
http://truffleframework.com/tutorials/robust-smart-contracts-with-openzeppelin

I install & open ganache to start the development network, the block height is zero;
Using 'truffle migrate' to deploy the contracts on the development network successfully, the block height in ganache is 4;
made a token transfer successfully, the block height in ganache is 5;
close the ganache and reopen ganache, the block height is reset to zero!
I redeploy the contracts,, the block height changed to 4;
made a token transfer failed, for the height is not corresponding.("Error: the tx doesn't have the correct nonce. account has nonce of:4 tx has nonce of:5 at runCall")
image
image

How can I fix this problem while using MetaMask to test?

@tmashuang
Copy link
Contributor

Hamburger icon, top-right, > Settings > Reset Account. This should only be used in the scenario that you are describing and not for any other network other than Ganache/Testrpc. Might require to disable/enable the extension.

@3for
Copy link
Author

3for commented Mar 20, 2018

Thank you so much. @tmashuang Reset the Account is just helpful.

@3for 3for closed this as completed Mar 20, 2018
github-merge-queue bot pushed a commit that referenced this issue Nov 21, 2024
## **Description**

Upgrade `@metamask/transaction-controller` to increase the pending
transaction polling rate.

[![Open in GitHub
Codespaces](https://github.com/codespaces/badge.svg)](https://codespaces.new/MetaMask/metamask-extension/pull/28452?quickstart=1)

## **Related issues**

Fixes:
[#3629](MetaMask/MetaMask-planning#3629)

## **Manual testing steps**

Regression of pending transaction polling including:

- Alternate Chains
- Queued Transactions
- Sequential Transactions
- Multiple Transactions

## **Screenshots/Recordings**

### **Before**

### **After**

## **Pre-merge author checklist**

- [x] I've followed [MetaMask Contributor
Docs](https://github.com/MetaMask/contributor-docs) and [MetaMask
Extension Coding
Standards](https://github.com/MetaMask/metamask-extension/blob/develop/.github/guidelines/CODING_GUIDELINES.md).
- [x] I've completed the PR template to the best of my ability
- [x] I’ve included tests if applicable
- [x] I’ve documented my code using [JSDoc](https://jsdoc.app/) format
if applicable
- [x] I’ve applied the right labels on the PR (see [labeling
guidelines](https://github.com/MetaMask/metamask-extension/blob/develop/.github/guidelines/LABELING_GUIDELINES.md)).
Not required for external contributors.

## **Pre-merge reviewer checklist**

- [ ] I've manually tested the PR (e.g. pull and build branch, run the
app, test code being changed).
- [ ] I confirm that this PR addresses all acceptance criteria described
in the ticket it closes and includes the necessary testing evidence such
as recordings and or screenshots.
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

No branches or pull requests

2 participants