Adding es-CO locale for mobile phones #1198
Conversation
index.js
Outdated
| @@ -1,5 +1,7 @@ | |||
| "use strict"; | |||
|
|
|||
| function _typeof(obj) { if (typeof Symbol === "function" && typeof Symbol.iterator === "symbol") { _typeof = function _typeof(obj) { return typeof obj; }; } else { _typeof = function _typeof(obj) { return obj && typeof Symbol === "function" && obj.constructor === Symbol && obj !== Symbol.prototype ? "symbol" : typeof obj; }; } return _typeof(obj); } | |||
There was a problem hiding this comment.
unstage the changes here, I know they were generated by your build system but remove them from your commit.
lib/isIP.js
Outdated
| @@ -9,6 +9,35 @@ var _assertString = _interopRequireDefault(require("./util/assertString")); | |||
|
|
|||
| function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; } | |||
|
|
|||
| /** | |||
There was a problem hiding this comment.
Also remove this file since it's unrelated.
| 'es-CL': /^(\+?56|0)[2-9]\d{1}\d{7}$/, | ||
| 'es-EC': /^(\+?593|0)([2-7]|9[2-9])\d{7}$/, |
There was a problem hiding this comment.
Did you add both es-CO and es-EC?
There was a problem hiding this comment.
Didn't check 'es-EC'. Seems like a spacing change.
There was a problem hiding this comment.
Actually, the es-EC was already introduced in src/lib/isMobilePhone.js. Perhaps checking with @mrcrow85?
|
And thanks for your contrib BTW 👍 |
zorrodg
force-pushed
the
feature/locale-es-co
branch
2 times, most recently
from
54e15e5 to
daccca2
Compare
|
@zorrodg -- could you fix the m/c and we should be good to go? Sorry for my delay to re-check this. |
profnandaa
added
mc-to-land
|
@zorrodg i think you should fix the merge conflicts. Rest looks good |
zorrodg
force-pushed
the
feature/locale-es-co
branch
from
daccca2 to
eb59416
Compare
|
Sorry it took so long. I stopped working on the project I needed this for, and quite recently picked it back up. Please let me know if anything else is needed. |
|
No worries, glad to see you back :) |
profnandaa
removed
the
mc-to-land
As titled. Please let me know if I need to add anything else. Thanks!