-
Notifications
You must be signed in to change notification settings - Fork 207
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
Refresh autopilot LS + forward #996
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.
very clean! looks good so far, pending the sender change!
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.
Looks great to me, thanks for doing this! Fwiw, I think this is also working on testnet now, but we can also add more connections there and test edge cases (e.g. sending to a new network)
@asalzmann fyi, unit tests for this are in #1041 |
Next steps for this PR
|
3b8680f
to
e87bdd5
Compare
(for the record)...
We decided to merge this first
Done and resolved comment
We decided it wasn't necessary |
Closes: #771
Context and purpose of the change
Add autopilot liquid stake + forward support. This allows autopilot liquid stakers to specify a receiving address and IBC channel, where stTokens will be forwarded after being liquid staked.
Open questions
Next steps
Brief Changelog
IbcReceiver
andTransferChannel
toStakeibcPacketMetadata
IBCTransferStAsset
function, called at the end ofRunLiquidStake
Test plan
IbcReceiver
andTransferChannel
fields