Skip to content

Use eth json rpc errors - #200

Merged
rickycodes merged 8 commits into
developfrom
use-eth-json-rpc-errors
Feb 26, 2020
Merged

Use eth json rpc errors#200
rickycodes merged 8 commits into
developfrom
use-eth-json-rpc-errors

Conversation

@rickycodes

@rickycodesrickycodes commented Feb 21, 2020

Copy link
Copy Markdown
Contributor

re: MetaMask/metamask-mobile#1314

I've confirmed that after this we get functionality similar to the extension in the mobile app:

image

todo

  • fix tests

@whymarrh

Copy link
Copy Markdown
Contributor

This might be aided by MetaMask/rpc-errors#9

@rickycodes
rickycodesforce-pushed the use-eth-json-rpc-errors branch from 741b0c6 to 5e0d6d9CompareFebruary 25, 2020 22:42
@codecov-io

Copy link
Copy Markdown

Codecov Report

Merging #200 into develop will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@ Coverage Diff @@## develop #200 +/- ##
======================================
Coverage 100% 100% ======================================
Files 23 23 Lines 1519 1520 +1 Branches 211 211 ======================================
+ Hits 1519 1520 +1
Impacted FilesCoverage Δ
src/transaction/TransactionController.ts100% <100%> (ø)⬆️
src/util.ts100% <100%> (ø)⬆️

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 95c42c1...5e0d6d9. Read the comment docs.

@rekmarksrekmarks left a comment

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.

LGTM!

@rickycodes
rickycodesforce-pushed the use-eth-json-rpc-errors branch from 5e0d6d9 to 1baf521CompareFebruary 25, 2020 23:14
@rickycodes
rickycodesforce-pushed the use-eth-json-rpc-errors branch from 1baf521 to 5365519CompareFebruary 25, 2020 23:17
@codecov-io

codecov-io commented Feb 25, 2020

Copy link
Copy Markdown

Codecov Report

Merging #200 into develop will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@ Coverage Diff @@## develop #200 +/- ##
======================================
Coverage 100% 100% ======================================
Files 23 23 Lines 1519 1520 +1 Branches 211 211 ======================================
+ Hits 1519 1520 +1
Impacted FilesCoverage Δ
src/transaction/TransactionController.ts100% <100%> (ø)⬆️
src/util.ts100% <100%> (ø)⬆️

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 95c42c1...5365519. Read the comment docs.

case 'failed':
return reject(new Error(meta.error!.message));
return reject(ethErrors.rpc.internal(meta.error!.message));
/* istanbul ignore next */

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.

This is fine!

@rickycodes
rickycodes merged commit d17d911 into developFeb 26, 2020
@rickycodes
rickycodes deleted the use-eth-json-rpc-errors branch February 26, 2020 03:48
This was referenced Feb 26, 2020
Sign up for freeto 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

@rickycodes@whymarrh@codecov-io@rekmarks