Skip to content
This repository was archived by the owner on Jun 27, 2022. It is now read-only.

Add filecoin cryptocurrency #651

Merged
merged 1 commit into from
Sep 16, 2021
Merged

Add filecoin cryptocurrency #651

merged 1 commit into from
Sep 16, 2021

Conversation

emmanuelm41
Copy link
Member

No description provided.

@codecov
Copy link

codecov bot commented Aug 30, 2021

Codecov Report

Merging #651 (cd1d3bd) into master (031cf51) will not change coverage.
The diff coverage is 83.33%.

❗ Current head cd1d3bd differs from pull request most recent head e00aa53. Consider uploading reports for the commit e00aa53 to get more accurate results
Impacted file tree graph

@@           Coverage Diff           @@
##           master     #651   +/-   ##
=======================================
  Coverage   40.93%   40.93%           
=======================================
  Files          74       74           
  Lines        4397     4397           
  Branches      720      720           
=======================================
  Hits         1800     1800           
  Misses       2580     2580           
  Partials       17       17           
Impacted Files Coverage Δ
packages/cryptoassets/src/currencies.ts 100.00% <ø> (ø)
packages/cryptoassets/src/tokens.ts 74.15% <50.00%> (ø)
packages/hw-app-eth/src/Eth.ts 85.84% <100.00%> (ø)
packages/hw-app-eth/src/erc20.ts 97.43% <100.00%> (ø)

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 031cf51...e00aa53. Read the comment docs.

@gre gre merged commit 9fbb234 into LedgerHQ:master Sep 16, 2021
@emmanuelm41
Copy link
Member Author

@gre I left a FIXME on the modified code. Do you want me to remove it?

@gre
Copy link
Contributor

gre commented Sep 17, 2021

@emmanuelm41 Hello, I've actually removed the line as I've checked against our API that all is fine with our countervalues for FIL :)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants