-
Notifications
You must be signed in to change notification settings - Fork 0
Add EURC and XLM assets from Pendulum #1
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
Releases: @galacticcouncil/[email protected]
Releases: @galacticcouncil/[email protected]
Releases: @galacticcouncil/[email protected]
Releases: @galacticcouncil/[email protected]
Releases: @galacticcouncil/[email protected]
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.
Please link to the related ticket in the description of this PR.
new AssetRoute({ | ||
source: { | ||
asset: eurc, | ||
balance: BalanceBuilder().substrate().system().account(), |
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.
Are you sure that this is correct? Only the native asset balance can be accessed in the system::account()
query so I guess this is wrong?
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.
You may be right, I didn't think about this and it wasn't important to create the transaction itself according to the tests, but it may be for the UI.
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.
Haha yeah I guess it's important, otherwise the balance won't show correctly.
chore: add license badge
Releases: @galacticcouncil/[email protected] @galacticcouncil/[email protected]
Releases: @galacticcouncil/[email protected]
Releases: @galacticcouncil/[email protected]
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 think it looks good to me. Feel free to not merge this PR but create a new PR to the upstream repository with the changes on this branch.
Thanks! Here is the PR to upstream. I think last step is to test once they provide the preview. And add the icons, but for that we need to confirm the id's are okay. |
Related issue: Tasks/#434