Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

fallback to placeholder when currency address is fetching #11771

Merged
merged 1 commit into from
Nov 3, 2017

Conversation

cezaraugusto
Copy link
Contributor

fix #11460
fix #11589

Input placeholder should be filled in case the wallet address is still fetching. Solution matches Brad spec.

screen shot 2017-11-02 at 4 38 12 pm

@codecov-io
Copy link

Codecov Report

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

@@           Coverage Diff           @@
##           master   #11771   +/-   ##
=======================================
  Coverage   52.75%   52.75%           
=======================================
  Files         269      269           
  Lines       25668    25668           
  Branches     4097     4097           
=======================================
  Hits        13542    13542           
  Misses      12126    12126
Flag Coverage Δ
#unittest 52.75% <ø> (ø) ⬆️
Impacted Files Coverage Δ
...ment/addFundsDialog/steps/addFundsWizardAddress.js 32.3% <ø> (ø) ⬆️
app/renderer/components/common/textbox.js 71.87% <ø> (ø) ⬆️

@NejcZdovc
Copy link
Contributor

@cezaraugusto not sure why this PR would close #11460?

@cezaraugusto
Copy link
Contributor Author

ya @bradleyrichter said so

@NejcZdovc
Copy link
Contributor

ok 😄

Copy link
Member

@bsclifton bsclifton left a comment

Choose a reason for hiding this comment

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

++

@bsclifton bsclifton merged commit e649693 into master Nov 3, 2017
@bsclifton bsclifton deleted the ca--11460 branch November 3, 2017 21:49
bsclifton added a commit that referenced this pull request Nov 3, 2017
fallback to placeholder when currency address is fetching
bsclifton added a commit that referenced this pull request Nov 3, 2017
fallback to placeholder when currency address is fetching
bsclifton added a commit that referenced this pull request Nov 3, 2017
fallback to placeholder when currency address is fetching
@bsclifton
Copy link
Member

master e649693
0.21.x d40a793
0.20.x 134f8ee
0.19.x 33e990e

@kjozwiak
Copy link
Member

kjozwiak commented Nov 9, 2017

Use case mentioned in #11589 (comment).

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.