facilityregistry / fred-api

Facility Registry API Documentation Website
11 stars 4 forks source link

Paging: rename limit=off to limit=0 #49

Closed mortenoh closed 11 years ago

mortenoh commented 11 years ago

I suggest this so that we have less confusion. Right now its hard to understand what limit=off means with regards to the offset parameter.

I suggest limit=0 means no limit.

sunbiz commented 11 years ago

+1

edjez commented 11 years ago

In general I don't like magic numbers - in this case Limit=0 means limit=infinite how about ?paging=false

edjez commented 11 years ago

From Morten: Also we need to specify what happens if you have limit=off, and then specify an offset, (ie return all records starting at offset)

mortenoh commented 11 years ago

I prefer paging=false to limit=off

edjez commented 11 years ago

Folks add comments with your opinions so we can rapidly close this one.

mberg commented 11 years ago

limit=off