Skip to content

Conversation

@pahor167
Copy link
Contributor

…Ledger Wallet

Add comprehensive documentation for the Celo Ledger app EIP-712 signing limitation and the Eth Recovery app workaround:

  • Create new dedicated guide (tooling/wallets/ledger/eip712-workaround.mdx)
  • Document the issue: Celo Ledger app doesn't support EIP-712 typed data
  • Explain this only affects users with Celo app + Celo derivation path
  • Provide step-by-step instructions for installing Eth Recovery app
  • Add derivation path configuration guidance
  • Include troubleshooting section and tested configurations

Update Ledger documentation to reflect current branding:

  • Rename all "Ledger Live" references to "Ledger Wallet" (98 occurrences)
  • Update Developer Mode instructions to match current Ledger UI
  • Add official Ledger developer documentation image

Add EIP-712 limitation warnings with links to workaround guide:

  • tooling/wallets/ledger/setup.mdx
  • tooling/wallets/ledger/to-celo-terminal.mdx
  • tooling/wallets/index.mdx
  • home/wallets.mdx

…Ledger Wallet

Add comprehensive documentation for the Celo Ledger app EIP-712 signing
limitation and the Eth Recovery app workaround:

- Create new dedicated guide (tooling/wallets/ledger/eip712-workaround.mdx)
- Document the issue: Celo Ledger app doesn't support EIP-712 typed data
- Explain this only affects users with Celo app + Celo derivation path
- Provide step-by-step instructions for installing Eth Recovery app
- Add derivation path configuration guidance
- Include troubleshooting section and tested configurations

Update Ledger documentation to reflect current branding:

- Rename all "Ledger Live" references to "Ledger Wallet" (98 occurrences)
- Update Developer Mode instructions to match current Ledger UI
- Add official Ledger developer documentation image

Add EIP-712 limitation warnings with links to workaround guide:

- tooling/wallets/ledger/setup.mdx
- tooling/wallets/ledger/to-celo-terminal.mdx
- tooling/wallets/index.mdx
- home/wallets.mdx
@pahor167 pahor167 requested review from a team as code owners January 28, 2026 12:41
@pahor167 pahor167 requested review from ezdac and palango and removed request for a team January 28, 2026 12:41
Copy link
Contributor

@Mc01 Mc01 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Small details to take care before merging 💅

…in EIP-712 workaround guide

Enhance the EIP-712 workaround documentation by including a note that the Eth Recovery app requires Developer Mode to be enabled in Ledger Wallet. This clarification helps users understand the necessary steps for proper functionality.
@pahor167 pahor167 merged commit df91291 into main Jan 29, 2026
5 checks passed
@pahor167 pahor167 deleted the pahor/celoAppWorkaround branch January 29, 2026 15:56
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

Successfully merging this pull request may close these issues.

3 participants