Skip to content
This repository has been archived by the owner on Nov 19, 2021. It is now read-only.

Include Planning IAs in metadata #1354

Open
GuiltyDolphin opened this issue May 12, 2016 · 8 comments
Open

Include Planning IAs in metadata #1354

GuiltyDolphin opened this issue May 12, 2016 · 8 comments
Assignees

Comments

@GuiltyDolphin
Copy link
Member

It would be really useful if we could include IAs in planning in the metadata - working with DuckPAN and using the metadata to fill out fields automatically kindah requires that we have access to planning IAs.

/cc @jdorweiler @zekiel

@jdorweiler
Copy link
Contributor

@zachthompson

@zachthompson
Copy link
Contributor

Yep, we've had a task for this for while.

@GuiltyDolphin
Copy link
Member Author

@zachthompson Just curious, but @jdorweiler and I had a bit of a discussion about this; the main issue here is that we may have a lot more Instant Answers in the metadata.

Are you thinking of having all the metadata together in the main JSON? Or perhaps have a separate file that contains everything? Either way, it'd be great to be able to access everything somehow.

@GuiltyDolphin
Copy link
Member Author

@zachthompson @jbarrett It looks like we can already access most of this through https://duck.co/ia/repo/all/json?all_milestones=1 ? Maybe we should just redirect where DDG::Meta::Data is fetching from?

@zachthompson
Copy link
Contributor

@GuiltyDolphin Not directly, there's a reason we aren't hitting duck.co.

@GuiltyDolphin
Copy link
Member Author

@zachthompson Hmm, okay - well that's the data I'm looking for, so maybe we can just package that up? We can always have some preset filters later on in Meta::Data (such as the set of live IAs).

@mintsoft
Copy link
Contributor

I'm tempted to say that we should either revert duckduckgo/zeroclickinfo-goodies#3018 or get this implemented as the pipeline for cheatsheets is now broken

@GuiltyDolphin
Copy link
Member Author

@mintsoft Maybe - a temp fix might be to only test metadata when an env variable is set (and set it in the Travis build); @zachthompson is this likely to be implemented soon?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants