-
Notifications
You must be signed in to change notification settings - Fork 101
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
IBC basic form #1106
IBC basic form #1106
Conversation
5a0ca67
to
56a9cec
Compare
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.
Really nice! This will help me a lot to integrate with the transfer module!
const [channelId, setChannelId] = useState(""); | ||
|
||
const withErrorReporting = | ||
(fn: () => Promise<void>): (() => Promise<void>) => |
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.
Can we make this signature more readable, or simplify this function?
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.
I've changed the function to use await
instead of then
.
b59b798
to
1509c8a
Compare
This is a very rough first draft intended to be used as a starting point for building more IBC features.
1509c8a
to
006a07e
Compare
This is a very rough first draft intended to be used as a starting point for building more IBC features.
006a07e
to
a47e2f7
Compare
This PR adds a very basic form for IBC transfers into Namada. It is a rough first draft intended to be used as a base for building further IBC features and is definitely not production-quality code. It only works for transfers from the main Cosmos testnet (
theta-testnet-001
).I've not used our existing
Keplr
class from the integrations package for this because I wanted to see how we need to use Keplr for IBC before building any abstractions. I was thinking I might end up refactoring the integrations in the end anyway because the existing integrations abstraction was written with the old interface in mind, which had a very different design.To test you will need gaiad and hermes set up locally. I can help anyone reviewing this PR set up what is needed.
Added
Fixed