Closed peterdesmet closed 9 years ago
@bartaelterman can you review and assign to @niconoe when OK.
Agreed.
Hi guys, that should be it! Please test the new API (at https://epu-index.herokuapp.com/api/epu/) and report any issue or close this ticket.
I tested it. Looks perfect.
@peterdesmet will you give it a final look and close this ticket?
Great, thanks!
Here's how I would change the EPU endpoint to serve for the charts (#2) as well as the downloads (#12):
Options
Changes:
start_date
andend_date
instead of less clearstart
andend
(especially since we'll have another endpoint with justdate
)Results
Changes:
number_of_newspapers
,number_of_articles
are required for csv download, and nice to have for json.number_of_papers
is renamed tonumber_of_newspapers
to avoid confusion with scientific "papers"id
is dropped