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] Fixed Native phone picker Use native API instead #1669

Merged
merged 5 commits into from
Nov 12, 2019

Conversation

martinvol
Copy link
Contributor

Description

react-native-device-info changed the result of it's getBaseOs() method so I switched to a react-native method.

Tested

Updated unitest

Other changes

Related issues

Backwards compatibility

@martinvol martinvol changed the title Fixed Native phone picker Use native API instead [wallet] Fixed Native phone picker Use native API instead Nov 12, 2019
@codecov
Copy link

codecov bot commented Nov 12, 2019

Codecov Report

Merging #1669 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1669   +/-   ##
=======================================
  Coverage   74.38%   74.38%           
=======================================
  Files         279      279           
  Lines        7566     7566           
  Branches      947      947           
=======================================
  Hits         5628     5628           
  Misses       1824     1824           
  Partials      114      114
Flag Coverage Δ
#mobile 74.38% <ø> (ø) ⬆️

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 110f856...1e4b749. Read the comment docs.

@martinvol martinvol added automerge Have PR merge automatically when checks pass wallet labels Nov 12, 2019
@martinvol martinvol merged commit 6bca7c3 into master Nov 12, 2019
@martinvol martinvol self-assigned this Nov 13, 2019
@martinvol martinvol deleted the martinvol/fix_native_phone_picker branch November 13, 2019 14:47
aaronmgdr added a commit that referenced this pull request Nov 14, 2019
* master: (56 commits)
  Adjust e2e transfer and governance tests to match new fee distribution and eliminate ProposerFraction (#1585)
  [Wallet] Add more local currencies (#1698)
  Switch to correct cluster when fauceting (#1687)
  [Wallet] Use the country of the phone number for determining the default local currency (#1684)
  [Wallet] Limit QR code scanner to 1 code per second (#1676)
  Update Dark backgrounds text color (#1677)
  Remove integration sync test
  Minor attestation service fixes (#1680)
  [wallet] Fixed Native phone picker Use native API instead (#1669)
  Fix token addresses for notification service (#1674)
  Add golang to setup docs
  [wallet] Hide invite education copy after invite was redeemed (#1670)
  [Wallet] Add spinner, timer, and tip text to Verification input screen (#1656)
  [Wallet] Fix app deprecation check mechanism (#1358)
  Point end-to-end governance test back to master (#1665)
  Add EpochRewards smart contract to calculate epoch rewards and payments (#1558)
  Optimized Attestation view calls and removal of the reveal TX (#1578)
  Support claim signatures and support Keybase claims (#1575)
  [Wallet] Add timestamp to top banner messages (#1657)
  Export geth metrics on VM testnet (#1351)
  ...

# Conflicts:
#	yarn.lock
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge Have PR merge automatically when checks pass wallet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants