Ønske: API returnerer verksinfo selv om det ikke finnes anmeldelser
boutros opened this issue · 7 comments
Eks: get http://datatest.deichman.no/api/reviews isbn=8205277362
returner
{
"error": "no reviews found"
}
som jo er riktig, det finnes ingen anbefalinger.
Men hvis API isteden returnerte følgende, kunne spørringen brukes til å søke opp isbn som bruker av anbefalinger.deichman.no ønsker å anmelde:
{
works [ {"title" :"Slaugtherhouse-five, The children's crusade"
"work_id":"http://data.deichman.no/work/xxxx......"
"cover_url": "http://krydder.bib.no/0156/785278.bilde.1327453183.s.jpg"
"author": "Kurt Vonnegut"
...etc
"reviews": []
]
}
Altså info som vanlig med bare med en tom [] works.first.reviews
- samme med work_id -> returnere verksinfo selv om ingen reviews blet funnet
pliiis?
makes sense ;) det må vi få til
håper du kan vente litt til... glemte at vi spør primært etter anbefalinger for å gjøre spørringen raskest mulig. dvs. vi går via anbefalinger for å finne verk, bok, etc.
hva med å lage et eget lite api/namespace for verk med isbn og verk som parametre foreløpig? Vi må jo trolig sy sammen flere apier etter hvert.
Aha, skjønner.
ingen hast, kan løse det på andre måter også, dvs med adapteret.
hva med å lage et eget lite api/namespace for verk med isbn og verk som parametre foreløpig?
Tenker du f.eks:
data.deichman.no/api/works uri=xx
data.deichman.no/api/works isbn =2134
Det hadde jo vært gull!
Nettopp.
I prinsippet burde de uansett vært separert slik at en slipper
krøssklipping i koden, som når jeg f.eks. må legge til rev:hasReview på
både verk og bok i en POST av review.
Men hva med responsen, hva trengs?
work -> uri, title, authors = [], isbns = [], books = [], reviews = []
noe annet?
2013/1/31 Petter Goksøyr Åsen notifications@github.com
hva med å lage et eget lite api/namespace for verk med isbn og verk som
parametre foreløpig?Tenker du f.eks:
data.deichman.no/api/works uri=xx
data.deichman.no/api/works isbn =2134Det hadde jo vært gull!
—
Reply to this email directly or view it on GitHubhttps://github.com//issues/5#issuecomment-12930808.
Enig
+cover_url?
hvis det finnes da