Skip to content

Feature(Eth) - Target ETH branch#9966

Merged
cmgustavo merged 1 commit into
bitpay:ethfrom
justinkook:eth
Sep 17, 2019
Merged

Feature(Eth) - Target ETH branch#9966
cmgustavo merged 1 commit into
bitpay:ethfrom
justinkook:eth

Conversation

@justinkook
Copy link
Copy Markdown
Contributor

@justinkook justinkook commented Jul 18, 2019

Requirements:

bitpay/bitcore#2281

Copay Set Up

Bitcore

  1. npm i in bitcore/ root directory.

  2. rm -rf node_modules package-lock.json in bitcore/packages/bitcore-wallet-client/ directory.

This is to remove symlinks / lerna bootstrap npm packages inside bitcore-wallet-client.

  1. npm i inside bitcore/packages/bitcore-wallet-client/ to install the NPM Registry versions of packages.

Copay

  1. rm -rf node_modules package-lock.json inside copay/ to remove old versions of bitcore-wallet-client.

  2. bash pack-all.sh inside copay/ to install local bitcore-wallet-client.

Status:

  • FIN

Steps to set up eth:

pack-all.sh assumes copay and bitcore folders to be a sibling and in the same folder.
ex:

dev/
   copay/
      src/
   bitcore/
      packages/
  1. Run pack-all.sh bash script to npm install your local version of bitcore-wallet-client into Copay. This is done using npm pack command. Update package versions as needed.

Your package.json should include this:

    "bitcore-wallet-client": "file:../bitcore/packages/bitcore-wallet-client/bitcore-wallet-client-8.6.0.tgz",
...
  1. Comment / Delete Content Security Policy in index-template.html:
 <!-- <meta http-equiv="Content-Security-Policy" content="default-src ... data:"> -->
  1. Point BWS url to http://localhost:3232/bws/api in bws.ts.

  2. When creating an ethereum wallet set BWS url to:
    http://localhost:3232/bws/api

Common Errors:

Uncaught TypeError: exists is not a function
Version guard duplicate version of bitcore-lib in bws.logs

Fix:
rm -rf node_modules/ && npm i

@codecov
Copy link
Copy Markdown

codecov Bot commented Jul 24, 2019

Codecov Report

Merging #9966 into eth will increase coverage by 0.13%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##              eth    #9966      +/-   ##
==========================================
+ Coverage   41.95%   42.08%   +0.13%     
==========================================
  Files         166      165       -1     
  Lines       15274    15166     -108     
  Branches     2105     2101       -4     
==========================================
- Hits         6408     6383      -25     
+ Misses       8503     8421      -82     
+ Partials      363      362       -1
Impacted Files Coverage Δ
src/pages/add/add.ts 38.88% <0%> (-16.12%) ⬇️
...ngs/wallet-group-settings/wallet-group-settings.ts 29% <0%> (-2.14%) ⬇️
...up-settings/wallet-group-name/wallet-group-name.ts
src/pages/home/home.ts 38.14% <0%> (+0.09%) ⬆️
src/providers/persistence/persistence.ts 53.61% <0%> (+0.24%) ⬆️
src/providers/profile/profile.ts 17.58% <0%> (+0.29%) ⬆️
src/providers/bwc/bwc.ts 87.5% <0%> (+0.4%) ⬆️
src/pages/settings/settings.ts 39.31% <0%> (+0.53%) ⬆️
src/providers/key/key.ts 77.56% <0%> (+1.93%) ⬆️
src/pages/add/select-currency/select-currency.ts 61.53% <0%> (+36.53%) ⬆️

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 313e9fd...f4945b0. Read the comment docs.

Comment thread src/components/clipboard-card/clipboard-card.html Outdated
@justinkook justinkook force-pushed the eth branch 2 times, most recently from 1422626 to 783772b Compare August 26, 2019 14:45
@justinkook justinkook mentioned this pull request Aug 29, 2019
@justinkook justinkook changed the title Eth Feature(Eth) - Target ETH branch Aug 29, 2019
@justinkook justinkook force-pushed the eth branch 2 times, most recently from ae8a54a to 7ceb786 Compare August 29, 2019 21:00
@justinkook justinkook mentioned this pull request Aug 29, 2019
Comment thread src/components/incoming-data-menu/incoming-data-menu.html Outdated
@justinkook justinkook force-pushed the eth branch 5 times, most recently from bf4f367 to 5d1437e Compare September 4, 2019 19:16
Comment thread src/components/incoming-data-menu/incoming-data-menu.html Outdated
Comment thread src/pages/integrations/shapeshift/shapeshift-confirm/shapeshift-confirm.ts Outdated
Comment thread src/pages/includes/multiple-outputs/multiple-outputs.html Outdated
Comment thread src/pages/send/confirm/confirm.html Outdated
Comment thread src/pages/send/confirm/confirm.ts Outdated
@justinkook justinkook force-pushed the eth branch 5 times, most recently from 11e17e0 to 60bfc87 Compare September 11, 2019 15:00
Comment thread src/pages/send/confirm/confirm.ts Outdated
@Gamboster
Copy link
Copy Markdown
Collaborator

code ACK! Really a great job. Just a couple of small comments

@justinkook justinkook force-pushed the eth branch 9 times, most recently from 884b35c to 1a0da25 Compare September 12, 2019 21:45
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.

3 participants