Skip to content
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

St 307 add guard clause to common x is undefined cases #287

Merged

Conversation

ericHgorski
Copy link
Contributor

No description provided.

@ericHgorski ericHgorski requested a review from mwmerz February 27, 2023 19:50
@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Feb 27, 2023

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: 1d8324d
Status: ✅  Deploy successful!
Preview URL: https://08c932b1.station-1a1.pages.dev
Branch Preview URL: https://st-307-add-guard-clause-to-c.station-1a1.pages.dev

View logs

@ericHgorski ericHgorski merged commit ab867ba into staging Feb 28, 2023
@ericHgorski ericHgorski deleted the ST-307-add-guard-clause-to-common-x-is-undefined-cases branch February 28, 2023 15:52
Copy link
Contributor

@mwmerz mwmerz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

Jared-TFL pushed a commit that referenced this pull request Feb 28, 2023
* fix: receive page bug on desktop (#264)

* Feat/loading optimizations (#267)

* fix: receive page bug on desktop

* wip: network optimization

* wip: network optimization 2

* wip: optimize InitChains comp

* wip: optimize InitNetworks  comp 2

* wip: optimize InitNetworks  comp 3

* wip: optimize InitNetworks  comp 4

* cleanup

* fix: loading bug

* fix: loading bug 2

* cleanup

* Fix network validation requests

* Add page loader during validation

---------

Co-authored-by: Alessandro Candeago <54709706+alecande11@users.noreply.github.com>

* hard coded stakes alliance tokens to 100 dollars (#269)

* feat: show disabled networks only after loading finishes (#275)

* ST-289 reset form (#280)

* feat: form resets on desktop after successful tx

* feat: submit send tx on enter key press

* cleanup

* St 286 donut graph fixes (#278)

* Made graph data switch from using value on individual chains to use amount to fix displaying 'NaN' when we dont have native token price. And changed display of '0% to '<1%' - ST-286

* fixed a lot of bug on the staking info cards. Made amounts show up on Delegations and Undelegations cards and other fixes - ST-286

* ST-293 fix token filter logic to not exclude assets without prices (#284)

* Fix lock function (#286)

* ST-248 improve addresses display/access in Connected component (#285)

* feat: receive page QR code

* feat: routing and component setup

* feat: address display changes on desktop

* feat: view address ledger button

* feat: style cleanup

* fix: bug to disconnect ledger

* St 307 add guard clause to common x is undefined cases (#287)

* fix: type error

* fix: guard close for prefix property

---------

Co-authored-by: Alessandro Candeago <54709706+alecande11@users.noreply.github.com>
Co-authored-by: Joshua Brigati <brigati.joshua@gmail.com>
Jared-TFL pushed a commit that referenced this pull request Mar 7, 2023
* fix: receive page bug on desktop (#264)

* Feat/loading optimizations (#267)

* fix: receive page bug on desktop

* wip: network optimization

* wip: network optimization 2

* wip: optimize InitChains comp

* wip: optimize InitNetworks  comp 2

* wip: optimize InitNetworks  comp 3

* wip: optimize InitNetworks  comp 4

* cleanup

* fix: loading bug

* fix: loading bug 2

* cleanup

* Fix network validation requests

* Add page loader during validation

---------

Co-authored-by: Alessandro Candeago <54709706+alecande11@users.noreply.github.com>

* hard coded stakes alliance tokens to 100 dollars (#269)

* feat: show disabled networks only after loading finishes (#275)

* ST-289 reset form (#280)

* feat: form resets on desktop after successful tx

* feat: submit send tx on enter key press

* cleanup

* St 286 donut graph fixes (#278)

* Made graph data switch from using value on individual chains to use amount to fix displaying 'NaN' when we dont have native token price. And changed display of '0% to '<1%' - ST-286

* fixed a lot of bug on the staking info cards. Made amounts show up on Delegations and Undelegations cards and other fixes - ST-286

* ST-293 fix token filter logic to not exclude assets without prices (#284)

* Fix lock function (#286)

* ST-248 improve addresses display/access in Connected component (#285)

* feat: receive page QR code

* feat: routing and component setup

* feat: address display changes on desktop

* feat: view address ledger button

* feat: style cleanup

* fix: bug to disconnect ledger

* St 307 add guard clause to common x is undefined cases (#287)

* fix: type error

* fix: guard close for prefix property

* fix: classic finder link (#293)

* fix: create sig bug (#289)

* feat: update tfm api url (#296)

* Changed Send title and back arrow to be absolute so content will scroll under it on smaller heights. Changed Receive page to have scroll on chain table with a max height (#292)

---------

Co-authored-by: Alessandro Candeago <54709706+alecande11@users.noreply.github.com>
Co-authored-by: Joshua Brigati <brigati.joshua@gmail.com>
Co-authored-by: Mike <17463738+mwmerz@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants