Hi I can’t Sell my NFT’s on Trezor it keeps showing “You declined the action in your wallet.” It already cost me a huge amount of gas fee. How can you help me?
forgi
February 25, 2022, 12:51pm
45
it is MM issue related to EIP712 which they have to implement, already discussed in this topic and on the forum.
MetaMask:develop
← MetaMask:trezor-eip-712-forked
opened 10:01PM - 20 Feb 22 UTC
**Pushing the branch of https://github.com/MetaMask/metamask-extension/pull/1335… 8 directly into the repo so that the MetaMask can move this forward over the next 24 hours.**
----
Description on the original PR (by @aloisklink):
Fixes: #11498
**Blocked by**:
- [ ] Merge and release of https://github.com/MetaMask/eth-trezor-keyring/pull/117
- [ ] Bumping `eth-trezor-keyring`'s version in `package.json`
Add EIP-712 `signTypedData_v4` support for Trezor.
Thanks to @alisinabh, @matejcik, and @BrandonNoad for their help getting Trezor Model 1 supported too in https://github.com/MetaMask/eth-trezor-keyring/pull/117, I've stuck you guys as co-authors in the git commit, let me know if you want me to remove you :wink:
### Manual testing steps:
- Go to https://metamask.github.io/test-dapp/
- Click on "Sign" and "Verify" for `signTypedData_v4` using:
- A Trezor Model T hardware wallet
- A Trezor Model 1 hardware wallet
It is not restricted to Polygon. I have been trying to list something on Ethereum. Plus when i checked the branch you posted, it says it’s been merged so the changes must be up, right?
Shouldn’t Trezor be following up with MM to get this up and running ASAP? This delay is literally costing Trezor holders money.
forgi
February 25, 2022, 1:02pm
47
it has been discussed here, Trezor supports eip712, we do not develop 3rd party apps, ask MM support on their forum: https://community.metamask.io/
edit: updates should be rolled out soon
What do we do for desktop?
Any idea of when we can see the changes on our Chrome extension? Do we need to do anything like clear cache?
forgi
February 26, 2022, 11:10am
51
browser extension is updated automatically, if you want to know when, you have to ask MM
Do you know if there is a thread I can follow from MM community forum on this topic?
1 Like
@accidentalsport - is it working for you now?
Nope, still not working @forgi
Cul2ure
February 28, 2022, 7:40pm
56
Hi @pavel - reading your response here and wondering if you can help. So before knowing much of this, I did set up a Trezor account in my MM and transferred some NFT’s. Do you have any updates or resources to simply help me get these NFT’s out of this account and back into an account that I can use to interact with Opensea and MM? Thanks for any updated resources you can provide.
Its almost 2 weeks, this is really slow development. People are losing money. I dont blame trezor they already do their part, its metamask now.
forgi
March 1, 2022, 12:56pm
58
@here the update from MM is still not implemented
MetaMask:master
← MetaMask:Version-v10.11.0
opened 08:08PM - 21 Feb 22 UTC
:package: :rocket:
## [10.11.0]
### Added
- Swaps: Add support for Smart Tr… ansactions on Mainnet and Rinkeby ([#12676](https://github.com/MetaMask/metamask-extension/pull/12676))
- Add "Token details" page ([#13216](https://github.com/MetaMask/metamask-extension/pull/13216))
- Add icons to sections in settings ([#12920](https://github.com/MetaMask/metamask-extension/pull/12920))
- Add EIP-712 support for Trezor ([#13693](https://github.com/MetaMask/metamask-extension/pull/13693))
### Changed
- Swaps: Change language "Quote X of Y" to "Fetching quote X of Y" ([#13663](https://github.com/MetaMask/metamask-extension/pull/13663))
- Update preferred account identicon selection UI ([#13408](https://github.com/MetaMask/metamask-extension/pull/13408))
- Include transactions from all networks in state logs ([#13599](https://github.com/MetaMask/metamask-extension/pull/13599))
- Update accounts connection language ([#13605](https://github.com/MetaMask/metamask-extension/pull/13605))
- Style updates for setting page in full screen mode ([#13569](https://github.com/MetaMask/metamask-extension/pull/13569))
- Add currency symbol validation in the add network form ([#12431](https://github.com/MetaMask/metamask-extension/pull/12431))
### Fixed
- Ensure pre-selected account on connect screen is visible in list ([#13621](https://github.com/MetaMask/metamask-extension/pull/13621))
- Do not allow transaction creation with gas limit below minimum ([#13574](https://github.com/MetaMask/metamask-extension/pull/13574))
- Fix network editing error when no block explorer is provided ([#13527](https://github.com/MetaMask/metamask-extension/pull/13527))
- Swaps: Fix BigNumber error when submitting quote ([#13555](https://github.com/MetaMask/metamask-extension/pull/13555))
i got the exactly same problem
Does anyone who has access to post topics in the Metamask channel post the question on when this PR will be merged and released to production:
MetaMask:master
← MetaMask:Version-v10.11.0
opened 08:08PM - 21 Feb 22 UTC
:package: :rocket:
## [10.11.0]
### Added
- Swaps: Add support for Smart Tr… ansactions on Mainnet and Rinkeby ([#12676](https://github.com/MetaMask/metamask-extension/pull/12676))
- Add "Token details" page ([#13216](https://github.com/MetaMask/metamask-extension/pull/13216))
- Add icons to sections in settings ([#12920](https://github.com/MetaMask/metamask-extension/pull/12920))
- Add EIP-712 support for Trezor ([#13693](https://github.com/MetaMask/metamask-extension/pull/13693))
### Changed
- Swaps: Change language "Quote X of Y" to "Fetching quote X of Y" ([#13663](https://github.com/MetaMask/metamask-extension/pull/13663))
- Update preferred account identicon selection UI ([#13408](https://github.com/MetaMask/metamask-extension/pull/13408))
- Include transactions from all networks in state logs ([#13599](https://github.com/MetaMask/metamask-extension/pull/13599))
- Update accounts connection language ([#13605](https://github.com/MetaMask/metamask-extension/pull/13605))
- Style updates for setting page in full screen mode ([#13569](https://github.com/MetaMask/metamask-extension/pull/13569))
- Add currency symbol validation in the add network form ([#12431](https://github.com/MetaMask/metamask-extension/pull/12431))
### Fixed
- Ensure pre-selected account on connect screen is visible in list ([#13621](https://github.com/MetaMask/metamask-extension/pull/13621))
- Do not allow transaction creation with gas limit below minimum ([#13574](https://github.com/MetaMask/metamask-extension/pull/13574))
- Fix network editing error when no block explorer is provided ([#13527](https://github.com/MetaMask/metamask-extension/pull/13527))
- Swaps: Fix BigNumber error when submitting quote ([#13555](https://github.com/MetaMask/metamask-extension/pull/13555))
forgi
March 2, 2022, 9:33am
61
@ABcrypto you should use their forum https://community.metamask.io/ , Github is for developers
hello? I see that like me, others also have the same problem every time I try to sell my NFT in opensea using etherium network, it automatically declines after clicking the sign button. There is no confirmation or whatsoever. I already paid for the one-time gas fee for listing NFTs and I can’t afford to transfer it to a new wallet and have to pay again for the one-time gas fee for listing again since it’s a new wallet. help us.
I commented on the Github PR and they looked at it and merged it
It should be working now. I was just able to list an item successfully.
Still not working for me. I get a blank white screen when I try to list. Have cleared cash, rebooted, etc, but no luck. Also can’t use collab.join to verify holdings on certain projects. Anything else I need to do to get the update working?