Merge pull request #59 from microservices-demo/payment-error-feedback

pidster · web-flow · commit 088aa7d0d7a1 · 2017-01-20T19:49:19.000Z
Improve error reporting
diff --git a/public/js/client.js b/public/js/client.js
@@ -100,11 +100,10 @@ function order() {
             }
         },
         error: function (jqXHR, textStatus, errorThrown) {
-            console.log('error: ' + JSON.stringify(jqXHR));
-            console.log('error: ' + textStatus);
-            console.log('error: ' + errorThrown);
+            response_payload = JSON.parse(jqXHR.responseText)
+            console.log('error: ' + jqXHR.responseText);
             if (jqXHR.status == 406) {
-                alert("Error placing order. Payment declined.");
+                alert("Error placing order. " +  response_payload.message);
             }
         }
     });

Original file line number	Diff line number	Diff line change
`@@ -100,11 +100,10 @@ function order() {`
`100`	`100`	`}`
`101`	`101`	`},`
`102`	`102`	`error: function (jqXHR, textStatus, errorThrown) {`
`103`		`- console.log('error: ' + JSON.stringify(jqXHR));`
`104`		`- console.log('error: ' + textStatus);`
`105`		`- console.log('error: ' + errorThrown);`
	`103`	`+ response_payload = JSON.parse(jqXHR.responseText)`
	`104`	`+ console.log('error: ' + jqXHR.responseText);`
`106`	`105`	`if (jqXHR.status == 406) {`
`107`		`- alert("Error placing order. Payment declined.");`
	`106`	`+ alert("Error placing order. " + response_payload.message);`
`108`	`107`	`}`
`109`	`108`	`}`
`110`	`109`	`});`