Skip to content

Fix: first load of wallet icons in wallets tab#11900

Merged
cmgustavo merged 1 commit intobitpay:masterfrom
Gamboster:fix/walletIconsFirstLoad
Sep 28, 2021
Merged

Fix: first load of wallet icons in wallets tab#11900
cmgustavo merged 1 commit intobitpay:masterfrom
Gamboster:fix/walletIconsFirstLoad

Conversation

@Gamboster
Copy link
Collaborator

No description provided.

@codecov
Copy link

codecov bot commented Sep 27, 2021

Codecov Report

Merging #11900 (eb6bee5) into master (01a0ed4) will decrease coverage by 0.00%.
The diff coverage is 0.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master   #11900      +/-   ##
==========================================
- Coverage   28.59%   28.59%   -0.01%     
==========================================
  Files         245      245              
  Lines       22066    22070       +4     
  Branches     3585     3586       +1     
==========================================
  Hits         6310     6310              
- Misses      15202    15206       +4     
  Partials      554      554              
Impacted Files Coverage Δ
src/pages/wallets/wallets.ts 15.74% <0.00%> (-0.52%) ⬇️

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 01a0ed4...eb6bee5. Read the comment docs.

@Gamboster Gamboster force-pushed the fix/walletIconsFirstLoad branch from bcd6760 to eb6bee5 Compare September 27, 2021 20:50
Copy link
Member

@cmgustavo cmgustavo left a comment

Choose a reason for hiding this comment

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

ACK

Copy link
Contributor

@rastajpa rastajpa left a comment

Choose a reason for hiding this comment

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

LGTM!

@cmgustavo cmgustavo merged commit 81018fc into bitpay:master Sep 28, 2021
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