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

fix: fix EIP-712 -> ERC-7730 conversion not using primary type #49

Merged
merged 2 commits into from
Oct 4, 2024

Conversation

jnicoulaud-ledger
Copy link
Contributor

@jnicoulaud-ledger jnicoulaud-ledger commented Oct 4, 2024

jnicoulaud-ledger added a commit to LedgerHQ/clear-signing-erc7730-registry that referenced this pull request Oct 4, 2024
@jnicoulaud-ledger jnicoulaud-ledger marked this pull request as ready for review October 4, 2024 14:02
@jnicoulaud-ledger jnicoulaud-ledger requested a review from a team as a code owner October 4, 2024 14:02
@jnicoulaud-ledger jnicoulaud-ledger merged commit 5a653b8 into main Oct 4, 2024
5 of 6 checks passed
@jnicoulaud-ledger jnicoulaud-ledger deleted the fix-converter branch October 4, 2024 14:32
jnicoulaud-ledger added a commit to LedgerHQ/clear-signing-erc7730-registry that referenced this pull request Oct 11, 2024
* import ledger-asset-dapps EIP-712 descriptors

* format descriptors

* update after LedgerHQ/python-erc7730#49 fix

* chore: update with erc7730 0.0.25

* merge 1inch descriptors

* merge degate descriptors

* merge dispatch descriptors

* merge ledgerquest descriptors

* merge lens descriptors

* merge opensea descriptors

* merge paraswap descriptors

* merge rarible descriptors

* merge smartcredit descriptors

* merge tally descriptors

* merge uniswap descriptors

* merge permit descriptors

* fixes/excluded

* fixes/excluded

* fixes/excluded

* fixes/excluded

* fixes/excluded

* fixes/excluded

* fixes/excluded

* add $schema
lcastillo-ledger added a commit to LedgerHQ/clear-signing-erc7730-registry that referenced this pull request Oct 12, 2024
* fix(paraswap): fix invalid path (#46)

* fix: typo (#51)

* import ledger-asset-dapps EIP-712 descriptors (#21)

* import ledger-asset-dapps EIP-712 descriptors

* format descriptors

* update after LedgerHQ/python-erc7730#49 fix

* chore: update with erc7730 0.0.25

* merge 1inch descriptors

* merge degate descriptors

* merge dispatch descriptors

* merge ledgerquest descriptors

* merge lens descriptors

* merge opensea descriptors

* merge paraswap descriptors

* merge rarible descriptors

* merge smartcredit descriptors

* merge tally descriptors

* merge uniswap descriptors

* merge permit descriptors

* fixes/excluded

* fixes/excluded

* fixes/excluded

* fixes/excluded

* fixes/excluded

* fixes/excluded

* fixes/excluded

* add $schema

* fix(AugustusSwapper): fix paths (#52)

* Add templates for ERC7730 files

---------

Co-authored-by: jnicoulaud-ledger <[email protected]>
jnicoulaud-ledger added a commit to LedgerHQ/clear-signing-erc7730-registry that referenced this pull request Oct 22, 2024
* fix(paraswap): fix invalid path (#46)

* fix: typo (#51)

* import ledger-asset-dapps EIP-712 descriptors (#21)

* import ledger-asset-dapps EIP-712 descriptors

* format descriptors

* update after LedgerHQ/python-erc7730#49 fix

* chore: update with erc7730 0.0.25

* merge 1inch descriptors

* merge degate descriptors

* merge dispatch descriptors

* merge ledgerquest descriptors

* merge lens descriptors

* merge opensea descriptors

* merge paraswap descriptors

* merge rarible descriptors

* merge smartcredit descriptors

* merge tally descriptors

* merge uniswap descriptors

* merge permit descriptors

* fixes/excluded

* fixes/excluded

* fixes/excluded

* fixes/excluded

* fixes/excluded

* fixes/excluded

* fixes/excluded

* add $schema

* fix(AugustusSwapper): fix paths (#52)

* Add templates for ERC7730 files

---------

Co-authored-by: jnicoulaud-ledger <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants