Skip to content

fix(swapbox): update native token balance correctly - #5511

Merged
guanbinrui merged 10 commits into
developfrom
fix/swapbox
Jan 25, 2022
Merged

fix(swapbox): update native token balance correctly#5511
guanbinrui merged 10 commits into
developfrom
fix/swapbox

Conversation

@UncleBill

@UncleBill UncleBill commented Jan 24, 2022

Copy link
Copy Markdown
Contributor

During cleaning up settings for Trader, I found out that balance of native token in Swapbox page is not correct.

Also fix icon color in TraderForm

@UncleBill UncleBill changed the title fix(swapbox): some small style issues fix(swapbox): update native token balance correctly Jan 24, 2022
Comment thread packages/mask/src/extension/popups/pages/Swap/index.tsx Outdated
Comment thread packages/mask/src/plugins/Trader/SNSAdaptor/trader/Trader.tsx
Comment thread packages/mask/src/plugins/Trader/SNSAdaptor/trader/hooks/useUpdateBalance.ts Outdated
@github-actions

github-actions Bot commented Jan 24, 2022

Copy link
Copy Markdown
Contributor

@github-actions
github-actions Bot temporarily deployed to pull request January 24, 2022 05:54 Inactive
Comment thread packages/mask/src/extension/popups/pages/Swap/SwapBox/index.tsx
Comment thread packages/mask/src/extension/popups/pages/Swap/index.tsx Outdated
Comment thread packages/mask/src/plugins/Trader/SNSAdaptor/trader/Trader.tsx
Comment thread packages/mask/src/plugins/Trader/SNSAdaptor/trader/hooks/useUpdateBalance.ts Outdated
@UncleBill
UncleBill requested a review from nuanyang233 January 24, 2022 07:37
@github-actions
github-actions Bot temporarily deployed to pull request January 24, 2022 07:52 Inactive
@github-actions
github-actions Bot temporarily deployed to pull request January 25, 2022 05:44 Inactive
inputAmount: '',
inputTokenBalance: '',
outputTokenBalance: '',
inputTokenBalance: '0x0',

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

0

Comment thread packages/mask/src/plugins/Trader/trader/useAllProviderTradeContext.ts Outdated
@github-actions
github-actions Bot temporarily deployed to pull request January 25, 2022 06:51 Inactive
@github-actions
github-actions Bot temporarily deployed to pull request January 25, 2022 07:16 Inactive
@guanbinrui
guanbinrui merged commit 1ba1490 into develop Jan 25, 2022
@guanbinrui
guanbinrui deleted the fix/swapbox branch January 25, 2022 07:36
etouyang added a commit to etouyang/Maskbook that referenced this pull request Jan 26, 2022
* 'develop' of github.com:etouyang/Maskbook:
  refactor: remove unused keys (DimensionDev#5523)
  feat: findtruman add property critical to votes (DimensionDev#5167)
  chore: remove unused settings for trader (DimensionDev#5516)
  feat: align collection data from nftscan and opensea (DimensionDev#5507)
  feat: trusted types again (DimensionDev#5514)
  chore: replace network icon style (DimensionDev#5519)
  fix(solana): load nft on dashboard (DimensionDev#5520)
  chore: fix button color (DimensionDev#5521)
  feat: implement mobile db methods with rpc (DimensionDev#5461)
  feat: simplify create persona (DimensionDev#5468)
  fix(swapbox): update native token balance correctly (DimensionDev#5511)
  feat: temporary slippage (DimensionDev#5459)
  fix(aurora): add explorer config (DimensionDev#5498)
  feat: connect mask (DimensionDev#5353)
  fix: claim all && nft wall (DimensionDev#5513)
  refactor: reduce rpc queries (DimensionDev#5393)
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.

4 participants