-
Notifications
You must be signed in to change notification settings - Fork 298
[LW-9778] Update cardano-node to 8.9.1 (via cardano-wallet) #3152
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Doskonale! 💪
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you :)
I tested this out on mainnet. I was able to see transaction history, wallet balances, etc... Slight issue with my trezor wallet that wants to set a spending password when I open it, but I checked the previous release and it has the same problem. I have not sent any transactions or done anything that requires signing anything. My suggestion would be merge it, and rebase #3156 on it. Then finish patching that one up and do as much manual testing as we can next couple weeks and hopefully get a P2P daedalus release out in middle of April. We can also involve the community in testing the release to get more assurance there aren't any edge cases we're missing. |
This PR updates cardano-node to 8.9.1
Todos
cardano-wallet
– v2024-03-27 •v2024-03-01cardano-launcher/src/cardanoNode.ts
to not force-bind the node to127.0.0.1
node_modules
in this repoScreenshots
n/a
Testing Checklist
Review Checklist
Basics
input-output-hk/daedalus-dev
andinput-output-hk/daedalus-qa
assigned as PR reviewersrelease-vNext
,feature
/bug
/chore
,WIP
)yarn manage:translations
produces no changes)yarn storybook
)yarn.lock
file is updatedCode Quality
Testing
After Review