Merge pull request #12 from torus-tools/publish_fixes

Publish fixes

lib/nameservers/update.js

@@ -25,6 +25,7 @@ function aws(domain, nameservers){
 //GoDaddy
 function godaddy(domain, nameservers){
   return new Promise((resolve, reject) => {
+    //console.log('EXECUTING NAMESERVER UPDATE ', domain, nameservers)
     var url = `https://api.godaddy.com/v1/domains/${domain}`
     axios({
       method: 'patch',
@@ -37,8 +38,8 @@ function godaddy(domain, nameservers){
         "nameServers": nameservers
       }
     })
-    .then(res => resolve(res.data))
-    .catch(err => reject(err))
+    .then(response => resolve(response.data))
+    .catch(error => reject(error))
   })
 }
 

lib/records/deleteAll.js

@@ -23,15 +23,18 @@ function aws(domain){
             )
           }
         }
-        var params = {
-          ChangeBatch: {
-            Changes: changes
-          }, 
-          HostedZoneId: id
+        if(changes.length > 0){
+          var params = {
+            ChangeBatch: {
+              Changes: changes
+            }, 
+            HostedZoneId: id
+          }
+          route53.changeResourceRecordSets(params).promise()
+          .then(data => resolve(data))
+          .catch(err => reject(err))
         }
-        route53.changeResourceRecordSets(params).promise()
-        .then(data => resolve(data))
-        .catch(err => reject(err))
+        else resolve('No records to delete')
       })
     }).catch(err=>reject(err))
   })

package.json

@@ -1,6 +1,6 @@
 {
   "name": "@torus-tools/domains",
-  "version": "0.0.12",
+  "version": "0.0.14",
   "description": "A standrad promise-based SDK for managing nameserver and DNS operations accorss multiple domain name registrars",
   "main": "index.js",
   "scripts": {

test/deleteAllRecordsTest.js

@@ -0,0 +1,5 @@
+const deleteAllRecords = require('../lib/records/deleteAll')
+
+deleteAllRecords.aws('localizehtml.com')
+.then(data => console.log(data))
+.catch(err => console.log(err))

test/updateNameserversTest.js

@@ -1,10 +1,12 @@
 const dns = require('../index')
 
 const ns = [
-  'ns-95.awsdns-54.net',
-  'ns-260.awsdns-43.com',
-  'ns-1873.awsdns-31.co.uk',
-  'ns-1050.awsdns-02.org'
+  'ns-822.awsdns-38.net',
+  'ns-19.awsdns-02.com',
+  'ns-1238.awsdns-26.org',
+  'ns-2034.awsdns-62.co.uk'
 ]
-dns.godaddy.updateNameServers('localizehtml.com', ns).then(data=>console.log(data)).catch(err=>console.log(err))
+
+dns.godaddy.updateNameservers('localizehtml.com', ns)
+.then(data=>console.log(data)).catch(err=>console.log(err))
 //updateNameServers.aws('torusframework.com', ns).then(data=>console.log(data)).catch(err=>console.log(err))