interlay / interbtc-ui

Kintsugi and Interlay App - BTC DeFi

https://app.interlay.io/

Apache License 2.0

32 stars 30 forks source link

Fixed a small dashboard display issue #1658

Closed mafux777 closed 3 months ago

mafux777 commented 6 months ago

Interbtc UI Pull Request Template

Description

Dashboard was crashing when certain data not available

To fix this, I added a question mark marking the variable as possibly undefined

vercel[bot] commented 6 months ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
interbtc-ui-interlay	✅ Ready (Inspect)	Visit Preview		Apr 5, 2024 0:40am
interbtc-ui-interlay-testnet	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Apr 5, 2024 0:40am
interbtc-ui-kintsugi	✅ Ready (Inspect)	Visit Preview		Apr 5, 2024 0:40am
interbtc-ui-kintsugi-testnet	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Apr 5, 2024 0:40am

github-actions[bot] commented 6 months ago

Running Lighthouse audit...

spazcoin commented 4 months ago

I looked at the unit test failures with @mafux777 today and don't see a relation between the failure and the change he submitted. I'm guessing the unit test failures are a package dependency instead.

tomjeatt commented 3 months ago

Merged—thanks for fixing! You're right, there are no unit test failures related to this PR but I appreciate you checking.