18F / fbopen

[DEPRECATED] An open API server, data import tools, and sample apps to help small businesses search for opportunities to work with the U.S. government.
Other
101 stars 45 forks source link

'p' param for paging not working #71

Closed arowla closed 10 years ago

aaronsnow commented 10 years ago

We should reconcile the existence of the (ahem, undocumented) p param with the start and limit params.

arowla commented 10 years ago

If we don't care about keeping around undocumented parameters, we might just want to go w/ the latter two. I was keeping p for backward compatibility, and because it was sort of nice to be able to just request pages relying on the default limit value.

aaronsnow commented 10 years ago

Yeah, it's pretty trivial to have both working. I suppose we should write a sentence in the doc clarifying which takes precedence if both p and start appear in the query. :-)