[GitHub] cordova-plugin-contacts pull request: [CB-11223] Better check for ...

2016-05-06 Thread TanaseButcaru
Github user TanaseButcaru commented on a diff in the pull request: https://github.com/apache/cordova-plugin-contacts/pull/125#discussion_r62412961 --- Diff: www/convertUtils.js --- @@ -28,10 +28,19 @@ module.exports = { */ toCordovaFormat: function (contact) {

[GitHub] cordova-plugin-contacts pull request: [CB-11223] Better check for ...

2016-05-06 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cordova-plugin-contacts/pull/125 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if th

[GitHub] cordova-plugin-contacts pull request: [CB-11223] Better check for ...

2016-05-06 Thread omefire
Github user omefire commented on a diff in the pull request: https://github.com/apache/cordova-plugin-contacts/pull/125#discussion_r62398237 --- Diff: www/convertUtils.js --- @@ -28,10 +28,19 @@ module.exports = { */ toCordovaFormat: function (contact) {

[GitHub] cordova-plugin-contacts pull request: [CB-11223] Better check for ...

2016-05-06 Thread omefire
Github user omefire commented on the pull request: https://github.com/apache/cordova-plugin-contacts/pull/125#issuecomment-217576715 LGTM! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have thi

[GitHub] cordova-plugin-contacts pull request: [CB-11223] Better check for ...

2016-05-06 Thread omefire
Github user omefire commented on a diff in the pull request: https://github.com/apache/cordova-plugin-contacts/pull/125#discussion_r62397911 --- Diff: www/convertUtils.js --- @@ -28,10 +28,19 @@ module.exports = { */ toCordovaFormat: function (contact) {

[GitHub] cordova-plugin-contacts pull request: [CB-11223] Better check for ...

2016-05-06 Thread TanaseButcaru
Github user TanaseButcaru commented on a diff in the pull request: https://github.com/apache/cordova-plugin-contacts/pull/125#discussion_r62396902 --- Diff: www/convertUtils.js --- @@ -28,10 +28,19 @@ module.exports = { */ toCordovaFormat: function (contact) {

[GitHub] cordova-plugin-contacts pull request: [CB-11223] Better check for ...

2016-05-06 Thread TanaseButcaru
Github user TanaseButcaru commented on a diff in the pull request: https://github.com/apache/cordova-plugin-contacts/pull/125#discussion_r62396643 --- Diff: www/convertUtils.js --- @@ -28,10 +28,19 @@ module.exports = { */ toCordovaFormat: function (contact) {

[GitHub] cordova-plugin-contacts pull request: [CB-11223] Better check for ...

2016-05-06 Thread TanaseButcaru
Github user TanaseButcaru commented on a diff in the pull request: https://github.com/apache/cordova-plugin-contacts/pull/125#discussion_r62396222 --- Diff: www/convertUtils.js --- @@ -28,10 +28,19 @@ module.exports = { */ toCordovaFormat: function (contact) {

[GitHub] cordova-plugin-contacts pull request: [CB-11223] Better check for ...

2016-05-06 Thread omefire
Github user omefire commented on a diff in the pull request: https://github.com/apache/cordova-plugin-contacts/pull/125#discussion_r62395371 --- Diff: www/convertUtils.js --- @@ -28,10 +28,19 @@ module.exports = { */ toCordovaFormat: function (contact) {

[GitHub] cordova-plugin-contacts pull request: [CB-11223] Better check for ...

2016-05-06 Thread omefire
Github user omefire commented on a diff in the pull request: https://github.com/apache/cordova-plugin-contacts/pull/125#discussion_r62395046 --- Diff: www/convertUtils.js --- @@ -28,10 +28,19 @@ module.exports = { */ toCordovaFormat: function (contact) {

[GitHub] cordova-plugin-contacts pull request: [CB-11223] Better check for ...

2016-05-06 Thread omefire
Github user omefire commented on the pull request: https://github.com/apache/cordova-plugin-contacts/pull/125#issuecomment-217565470 Do you know why is iOS setting an 'Invalid Date' in the first place ? I feel like this fix might end up just masking an issue on the native side. ---

[GitHub] cordova-plugin-contacts pull request: [CB-11223] Better check for ...

2016-05-06 Thread TanaseButcaru
Github user TanaseButcaru commented on a diff in the pull request: https://github.com/apache/cordova-plugin-contacts/pull/125#discussion_r62394372 --- Diff: www/convertUtils.js --- @@ -28,10 +28,19 @@ module.exports = { */ toCordovaFormat: function (contact) {

[GitHub] cordova-plugin-contacts pull request: [CB-11223] Better check for ...

2016-05-06 Thread omefire
Github user omefire commented on a diff in the pull request: https://github.com/apache/cordova-plugin-contacts/pull/125#discussion_r62393819 --- Diff: www/convertUtils.js --- @@ -28,10 +28,19 @@ module.exports = { */ toCordovaFormat: function (contact) {

[GitHub] cordova-plugin-contacts pull request: [CB-11223] Better check for ...

2016-05-06 Thread TanaseButcaru
GitHub user TanaseButcaru opened a pull request: https://github.com/apache/cordova-plugin-contacts/pull/125 [CB-11223] Better check for date validity of contact.birthday / Fix: RangeError Invalid Date You can merge this pull request into a Git repository by running: $ git pul