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 ed25519 verification #1909

Merged
merged 1 commit into from Oct 27, 2022
Merged

fix ed25519 verification #1909

merged 1 commit into from Oct 27, 2022

Conversation

librelois
Copy link
Collaborator

What does it do?

What important points reviewers should know?

Is there something left for follow-up PRs?

What alternative implementations were considered?

Are there relevant PRs or issues in other repositories (Substrate, Polkadot, Frontier, Cumulus)?

What value does it bring to the blockchain users?

@librelois librelois added B5-clientnoteworthy Changes should be mentioned in any downstream projects' release notes D5-nicetohaveaudit⚠️ PR contains trivial changes to logic that should be properly reviewed. labels Oct 27, 2022
@librelois librelois marked this pull request as ready for review October 27, 2022 16:03
@librelois librelois merged commit 739a11a into master Oct 27, 2022
@librelois librelois deleted the elois-substrate-fix-ed25519 branch October 27, 2022 16:03
imstar15 pushed a commit to OAK-Foundation/moonbeam that referenced this pull request May 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
B5-clientnoteworthy Changes should be mentioned in any downstream projects' release notes D5-nicetohaveaudit⚠️ PR contains trivial changes to logic that should be properly reviewed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants