RobertSpread / spreadcart

simple plugin to display a cart for the spreadshop everywhere on your domain
1 stars 1 forks source link

Support for node.js + ajax error handling #11

Closed jtlapp closed 8 years ago

jtlapp commented 8 years ago

These mods mainly support the addition of a node.js proxy, but they also improve client error handling. You might make sure I didn't break the PHP side of things.

(1) Added a node.js proxy server, including a barebones express server server.js and a single route cart.js. The route can also be plugged into existing node.js servers. The server only supports item deletion at the moment.

(2) Added proxyPath configuration parameter.

(3) Added operation proxy service parameter.

(4) Sorted methods in spreadCart.js by category to facilitate locating them.

(5) Now only updates cart data for deletion upon successful deletion.

(6) Proxy errors are reported via meaningful alerts. The server may return a JSON message parameter for error status 500, which the client displays.

(7) Made the delete-item text look like a clickable link, and turned the cursor into a pointer (as with links) when hovering over it.

(8) Tried to make the documentation more comprehensive.

I'll be working on the ability to change quantities next.

RobertSpread commented 8 years ago

Since I am not a developer at all, I am starting to lose control over this here. I honestly do not understand most of it by now :) Nevertheless, I will merge this pull request based on trust and your motivation. I just fear that I can't be of much help in the future.