935122 - Implement node's `querystring`

Status: RESOLVED INCOMPLETE

(Add-on SDK Graveyard :: General, defect, P3)

Description

[Jordan Santell [:jsantell] [@jsantell]]

http://nodejs.org/api/querystring.html

Comment 1

[Erik Vold [:erikvold] (please needinfo? me)]

Jordan what is needed that is not in https://github.com/mozilla/addon-sdk/commits/master/lib/sdk/querystring.js ?

Comment 2

[Jordan Santell [:jsantell] [@jsantell]]

Doesn't look like we're missing anything that Node API has -- these tickets are just a list of APIs that need "ported" over to be consumed via Node (some need created, some are just aliases of what we already have), whether that means a location change or what, so this one will be relatively simple

Comment 3

[Jordan Santell [:jsantell] [@jsantell]]

Attachment: GH PR 1336

Leaving deprecation of sdk/querystring for a future step once we have native loader

Comment 4

[Jordan Santell [:jsantell] [@jsantell]]

Comment on attachment 8357903 [details] [review]
GH PR 1336

Removing this r? until we decide what to do with these shims

Comment 5

[Jordan Santell [:jsantell] [@jsantell]]

Unassigning myself from bugs I haven't gotten around to

Comment 6

[Firefox Bug Husbandry Bot]

https://bugzilla.mozilla.org/show_bug.cgi?id=1399562