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

[Wallet] Update account amount overview style #1325

Merged
merged 6 commits into from
Oct 23, 2019

Conversation

@Pedro-vk Pedro-vk force-pushed the feature/1261-main-currency-style branch from 915c785 to 2f5bd21 Compare October 16, 2019 13:04
@jmrossy jmrossy self-assigned this Oct 18, 2019
@jmrossy jmrossy assigned Pedro-vk and unassigned jmrossy Oct 18, 2019
@Pedro-vk Pedro-vk force-pushed the feature/1261-main-currency-style branch 3 times, most recently from cc7409e to 66af9d0 Compare October 22, 2019 21:28
@Pedro-vk Pedro-vk force-pushed the feature/1261-main-currency-style branch from 66af9d0 to 9aeaf6b Compare October 22, 2019 23:45
@Pedro-vk Pedro-vk requested a review from jmrossy October 22, 2019 23:45
@Pedro-vk Pedro-vk force-pushed the feature/1261-main-currency-style branch from 9aeaf6b to 55a963e Compare October 23, 2019 08:06
@codecov
Copy link

codecov bot commented Oct 23, 2019

Codecov Report

Merging #1325 into master will increase coverage by 0.05%.
The diff coverage is 81.92%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1325      +/-   ##
==========================================
+ Coverage   67.62%   67.68%   +0.05%     
==========================================
  Files         269      269              
  Lines        7984     8023      +39     
  Branches      539      553      +14     
==========================================
+ Hits         5399     5430      +31     
- Misses       2477     2479       +2     
- Partials      108      114       +6
Flag Coverage Δ
#mobile 67.68% <81.92%> (+0.05%) ⬆️
Impacted Files Coverage Δ
packages/mobile/src/transactions/actions.ts 52.5% <ø> (ø) ⬆️
packages/mobile/src/components/CurrencyDisplay.tsx 100% <100%> (ø) ⬆️
packages/mobile/src/localCurrency/selectors.ts 73.07% <100%> (+3.51%) ⬆️
packages/mobile/src/localCurrency/hooks.ts 100% <100%> (ø) ⬆️
packages/mobile/src/localCurrency/consts.ts 100% <100%> (ø) ⬆️
packages/mobile/src/home/CeloDollarsOverview.tsx 88.46% <62.5%> (-11.54%) ⬇️
...kages/mobile/src/send/TransferConfirmationCard.tsx 81.63% <70.37%> (-1.71%) ⬇️
...kages/mobile/src/transactions/TransferFeedItem.tsx 82.45% <80%> (-1.19%) ⬇️
...kages/mobile/src/transactions/ExchangeFeedItem.tsx 88.46% <84.61%> (-1.79%) ⬇️
...s/mobile/src/exchange/ExchangeConfirmationCard.tsx 96.77% <93.33%> (+0.47%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b07c091...55a963e. Read the comment docs.

@jmrossy jmrossy removed their assignment Oct 23, 2019
@jmrossy jmrossy merged commit 2504bc9 into celo-org:master Oct 23, 2019
tkporter pushed a commit that referenced this pull request Oct 24, 2019
* Show the local currency amount as primary amount and then Celo dollars
* Use local currencies in more screens, including drilldown and exchange
aaronmgdr added a commit that referenced this pull request Oct 24, 2019
* master:
  [Wallet] Wallet can switch between hosted and local node (#1419)
  [Wallet] Prevent error from Avatar when name is missing (#1454)
  [Wallet] Show splash screen until JS is ready on iOS (#1453)
  Use new segment api keys used by both iOS and Android (#1452)
  [Wallet] Don't log all props, which includes i18n (#1445)
  [Helm] Updated the helm package to deploy the upgraded blockscout version (#1129)
  Tiny copy change (#1429)
  [contractkit] SortedOraclesWrapper + tests (#1405)
  [wallet] Refactor leftover thunk to sagas (#1388)
  [Wallet] Fix repeated QR code scanning and related navigation issues (#1439)
  [Wallet] Show the currency values with correct rounding. (#1435)
  [Wallet] Fix firebase initialization error on iOS after reinstalling the app (#1423)
  [Wallet] Use exit on iOS since we can't restart like Android (#1424)
  [Wallet] Update local currency styles and layout (#1325)
  Reset pincode cache if unlock fails (#1430)
jmrossy pushed a commit that referenced this pull request Oct 25, 2019
* Add react hooks lint rule to prevent incorrect usage of hooks
* Fix local currency regressions from #1325 
* Fix crash when changing local currency from the settings due to hooks incorrectly used
* Fix dollar exchange rate value shown in local currency in the exchange confirmation card
* Fix received gold amounts shown in the local currency
* Fix gold received transaction showing up as local currency in the full screen card details
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.

Users SBAT see switched currency order on home screen
2 participants