search

norfolkjs / norfolkjs.github.io

Website code for Norfolk.js
http://www.norfolkjs.org
12 stars 19 forks source link

limit events to only next two months #63

Closed qwo closed 10 years ago

qwo commented 10 years ago

another stanley issue resolved #62

lynnaloo commented 10 years ago

You can't restrict the dates in the query? It looks like they have a time in here. I haven't tried it though.

qwo commented 10 years ago

ah.. momentjs handled this. rewriting the query requires resigning the key and i didn't think about it.

isn't it cool to solve more problems in JS? :smiley:

lynnaloo commented 10 years ago

So this is basically like querying a database, right? You wouldn't write JS to limit your results from a database query, you'd change your querystring.

tjwebb commented 10 years ago

isn't it cool to solve more problems in JS?

Meetup API is willing to do the work for you. Code that is not necessary, regardless of how ostensibly clever, is wrong.

qwo commented 10 years ago

this has been fixed @lynnaloo maybe pull?

tjwebb commented 10 years ago

Looks good. Why did sig_id change?

qwo commented 10 years ago

i had to change query parameters so it regenerated a new signed request so a different sig_id