-
Notifications
You must be signed in to change notification settings - Fork 435
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: variable names and event handling logic #2421
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
@sds Hey! Resolved conflicts and opened a new PR. |
Looks like tests are failing. |
@sds How can I solve it? |
Look at the It appears that there are now two declarations of the same method ( |
db162e3
to
3e84957
Compare
3e84957
to
5c09792
Compare
Why is this change needed?
Describe why this issue should be fixed and link to any relevant design docs, issues or other relevant items.
Merge Checklist
Choose all relevant options below by adding an
x
now or at any time before submitting for reviewPR-Codex overview
This PR focuses on correcting typos in function and variable names across multiple files, specifically related to "verifications" and "multi addresses". It also ensures consistent naming conventions throughout the codebase.
Detailed summary
solanaVerficationsEnabled
tosolanaVerificationsEnabled
.getAllVerficationMessagesByFid
togetAllVerificationMessagesByFid
.processRustCommitedTransaction
toprocessRustCommittedTransaction
.ConnectToMutliAddr
toConnectToMultiAddr
in various locations.