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

Add information regarding Ed25519 & ECDSA Keys to docs (fixed verification) #430

Merged
merged 3 commits into from
Feb 18, 2025

Conversation

itsbrandondev
Copy link
Contributor

This PR adds a new page to the WalletConnect documentation that explians the differences of Ed25519 & ECDSA keys, important for ETH developers who need full EVM compatibility.

It also modifies one page to move it down the sidebar.

This PR is informational and only adds to the documentation, it does not impact the library.

Note for reviewers: this is a new version of PR 419 (my github gpg verification was not set up)

@itsbrandondev itsbrandondev added the New Feature A new feature, service, or documentation. Major changes that are not backwards compatible. label Feb 3, 2025
@itsbrandondev itsbrandondev requested review from a team as code owners February 3, 2025 21:44
Copy link

github-actions bot commented Feb 3, 2025

Coverage report

St.
Category Percentage Covered / Total
🟡 Statements 78.65% 523/665
🟡 Branches 64.34% 92/143
🟡 Functions 77.63% 118/152
🟡 Lines 79.94% 494/618

Test suite run success

153 tests passing in 11 suites.

Report generated by 🧪jest coverage report action from 78829d2

@itsbrandondev
Copy link
Contributor Author

i will be applying new updates now that the PR is fixed

tmctl
tmctl previously approved these changes Feb 4, 2025
@tmctl tmctl merged commit bb5a5d0 into main Feb 18, 2025
12 checks passed
@tmctl tmctl deleted the ed-ec-v3 branch February 18, 2025 15:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
New Feature A new feature, service, or documentation. Major changes that are not backwards compatible.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants