Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Use the default node.js query string library #4

Closed
wants to merge 2 commits into
from

Conversation

Projects
None yet
1 participant

jamiter commented Feb 21, 2014

The qs library which was used filtered out stuff like isVatExempt=1. In the query it only left &isVatExempt, without any value. Cheddargetter didn't understand that a customer was now vatExempt.

The only real change is:

-var qs = require("qs");
+var qs = require("querystring");

jamiter commented Feb 21, 2014

Never mind. This removes nesting support... I think we simply need to update the qs version to solve this. I'll do a new pull request.

@jamiter jamiter closed this Feb 21, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment