These are chat archives for cltk/cltk_api

28th
Nov 2015
Luke Hollis
@lukehollis
Nov 28 2015 15:23
Hey these look great!
Okay, perfect--I'll start using the frontend to access these
Would anyone be against merging the ingest branch with master now?
We have to do JSONP to get the data from the API server
Luke Hollis
@lukehollis
Nov 28 2015 15:29
But I can cache the results from these queries with Homer really easily in Mongo and then expose them via our Query resource
Kyle P. Johnson
@kylepjohnson
Nov 28 2015 15:29
Luke, you should have permissions to merge. Go for it!
Luke Hollis
@lukehollis
Nov 28 2015 15:29
Can also standup our frontend app on the shared server when we're ready
okay, awesome, will do!
Kyle P. Johnson
@kylepjohnson
Nov 28 2015 15:30
Today, as soon as I can, I will re-implement the /authors and /texts routing, which currently isn't working.
Also, what is JSONP?
Luke Hollis
@lukehollis
Nov 28 2015 15:31
Okay, good deal--I'll merge right now then
Luke Hollis
@lukehollis
Nov 28 2015 15:43
The only real difference is the the extra jsonp decorator for some of the queries: https://github.com/cltk/cltk_api/blob/master/util/jsonp.py
Kyle P. Johnson
@kylepjohnson
Nov 28 2015 15:53
So should I add @jsonp to each flask function?
I'll let you know how the scraper comes ... my first priority is to get you some good data to build off. Even a successful scrape will require some hand correction
Kyle P. Johnson
@kylepjohnson
Nov 28 2015 20:05
Some API improvements:
@lukehollis please take a look and let me know any changes you want. Also let me know about the @jsonp wrapper :)