-
Notifications
You must be signed in to change notification settings - Fork 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
495 enable circle eurcs as spacewalk asset on the portal on pendulum #525
495 enable circle eurcs as spacewalk asset on the portal on pendulum #525
Conversation
✅ Deploy Preview for rococo-souffle-a625f5 ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
* improve code readability * extract mykobo logic into handleSpecialAsset 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.
LGTM 🚀 ✅
@Sharqiewicz I noticed that all EURC icons are now showing the Circle EURc though I was not yet able to pin down why. This happens in the dashboard as well as the asset selector of the spacewalk dialogs. Very weird. |
No asset matches the Mykobo issuer address, I am resolving it |
I also thought that something was wrong with that match but actually, if you add plenty of log statements you will see that sometimes, it is able to match it. I think the I was not able to find the places where the issuer is not passed though. |
@ebma The issue was that we were invoking |
Ahh I see, thanks a lot for debugging and fixing this @Sharqiewicz! 🙏 I'll merge it then. |
Refactors the logic to get an icon so that it checks using
includes
and not for perfect matches and adds an additional check to decide between the mEURC and cEURC icons.With these changes, the Circle EURC icon should show properly in the dashboard as well as the asset selector in the Spacewalk menus. We don't care about Nabla yet as that's a special use case where we don't need to have a perfect solution just yet.
Closes #495.