Following the https://github.com/LedgerHQ/clear-signing-erc7730-registry/blob/master/specs/erc-7730.md I added the ERC4626 capabilities so that all DeFi vaults following this standard can now be clear-signed on ledger devices
https://github.com/LedgerHQ/clear-signing-erc7730-registry/pull/86 - Adding ERC-4626 standard parsing
It’s currently complete to test the clear singing on device using the current https://app.devicesdk.ledger-test.com/signer/ethereum, it’s unclear how to test on device with just a transaction field as the current workshop example only cover EIP-712 message singing, I tried to build the etherjs v6 transaction object locally but I couldn’t trigger the transaction on device, the webpage loaded without any response
The current PR is confirmed and “green” on the github CI but additional approval are required on the CAL review the contract informations in https://app.devicesdk.ledger-test.com/signer/ethereum
The current EIP-7730 should be updated to simplify the management of registry containing multiple deployment. For example protocols like Kiln or Yearn, have many deployments (from 10 to 100s), they will need a simple way to update this list overtime
contract.deployments.[0]
.."params": {
"tokenPath": "$.context.contract.deployments.[0].address"
}
In my opinionJSON
is not a adapted format for collaboration and peer review. I recommend exploring a format where we can leave comments
"_comment": "",
While the developer-preview
is great to quickly iterate and confirm the formatting on device, it require improvements so we can test with example calldata locally