-
Notifications
You must be signed in to change notification settings - Fork 68
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1107 +/- ##
======================================
Coverage 100% 100%
======================================
Files 174 174
Lines 6518 6518
Branches 1149 1149
======================================
Hits 6518 6518
Continue to review full report at Codecov.
|
@@ -7,7 +7,7 @@ describe('Phone Field directive', function () { | |||
selectors = { | |||
flagContainer: '.flag-container', | |||
countryList: '.country-list', | |||
localCountryTextbox: 'input[placeholder="(201) 555-5555"]', | |||
localCountryTextbox: 'input[placeholder="(201) 555-0123"]', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
intl-tel-input
changed the default placeholder in its recent versions.
@@ -34,7 +34,7 @@ | |||
// So, its dial code should just be 1 because the area code includes the dial code. | |||
// Example countries: Bahamas, Cayman Islands, Barbados. | |||
if (selectedCountryData.dialCode.toString()[0] === '1') { | |||
selectedCountryData.dialCode = 1; | |||
selectedCountryData.dialCode = '1'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a logical error in the directive where it was setting the phone number dial code to an integer instead of a string. This had to be remedied to use the newer version of the library.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Confirmed this fixes the two issues reported when running locally. Ship it!
Original contribution: blackbaud/skyux#1105
Issue: blackbaud/skyux#1104
Docs: N/A