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] Fix yarn dev command for running android #1534

Merged
merged 5 commits into from
Oct 30, 2019
Merged

Conversation

jmrossy
Copy link
Contributor

@jmrossy jmrossy commented Oct 30, 2019

Description

yarn dev is broken since the RN upgrade to 0.61
This is basically because react-native run-android runs the bundler from the monorepo root instead of the mobile project. This is a simple workaround for now.

@codecov
Copy link

codecov bot commented Oct 30, 2019

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff            @@
##           master    #1534    +/-   ##
========================================
  Coverage   73.74%   73.74%            
========================================
  Files         277      277            
  Lines        7420     7420            
  Branches      954      660   -294     
========================================
  Hits         5472     5472            
  Misses       1836     1836            
  Partials      112      112
Flag Coverage Δ
#mobile 73.74% <ø> (ø) ⬆️

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 440bee2...e83a86d. Read the comment docs.

Copy link
Contributor

@jeanregisser jeanregisser left a comment

Choose a reason for hiding this comment

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

Ship it 👍 🚢

@jmrossy jmrossy force-pushed the rossy/wa-metro-fix branch from 3288a1a to bb114c9 Compare October 30, 2019 14:00
@i1skn
Copy link
Contributor

i1skn commented Oct 30, 2019

Just one little downside - when app loads for the first time packager is not ready yet and you need to reload the app.

@jmrossy
Copy link
Contributor Author

jmrossy commented Oct 30, 2019

Just one little downside is that the first time the app loads - packager is not ready yet and you need to reload the app.

Strange, for me I don't need to reload the app. I just need to wait for the bundler to be ready

@jmrossy jmrossy added the automerge Have PR merge automatically when checks pass label Oct 30, 2019
@jmrossy jmrossy merged commit 4fc2ddf into master Oct 30, 2019
@jmrossy jmrossy deleted the rossy/wa-metro-fix branch October 30, 2019 16:25
aaronmgdr added a commit that referenced this pull request Nov 2, 2019
* master: (62 commits)
  Fix e2e on CI (#1537)
  Allow a specified address to disable/enable the Exchange  (#1467)
  Avoid re-encrypting key files with yarn keys:encrypt command (#1560)
  Support protocol hotfixing (#613)
  Point e2e tests back (#1562)
  Refactor to Accounts.sol (#1392)
  Add selectIssuers Transaction (#1327)
  [Wallet] Get React Native Hot Reloading Working (#1551)
  Unify to prefix messages for signing (#1473)
  [Wallet] Improve error handling around account creation and keystore ops (#1497)
  Add CI test for checking licenses and misc package.json cleanup (#1550)
  [Wallet] Implement SMS invite on iOS (#1541)
  CI: brings back to master (#1554)
  Validators: uses Ethereum address for proof of possession (#1494)
  Validate Attestation Requests (#1468)
  Rename hosted node references to forno (#1511)
  Bump rubyzip from 1.2.3 to 1.3.0 in /packages/mobile (#1508)
  Added txpool family to geth apis. Sorted geth cmd options (#1462)
  [Wallet] Fix yarn dev command for running android (#1534)
  [Wallet] iOS info plist changes and version bump (#1539)
  ...

# 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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants