search

pyk / rojak

Project Rojak, Yuk gabung diskusi di Slack nya Rojak: http://team.rojak.id/
Other
98 stars 46 forks source link

[rojak-api] Spesifikasi API v0.3 dan implementasi #185

Closed bobbypriam closed 7 years ago

bobbypriam commented 7 years ago

Spesifikasi dan implementasi berdasarkan skema data v0.5 dan draft spesifikasi API v0.3.

Spesifikasi saat ini bisa dilihat pada: http://rojak-api-spec-staging.surge.sh/ Implementasi API v0.3.0-beta sudah live di https://api.rojak.id/v1/

Monggo reviewnya 🙇 Kalo udah oke nanti jangan langsung merge dulu ya, di-bump ke v0.3.0 dulu hehe.

pyk commented 7 years ago

@bobbypriambodo btw untuk yg https://api.rojak.id/v1/news?embed[]=sentiments itu ada kasus dimana news belum punya sentiment. Kalo kita tampilin yang punya sentiment aja gimana?

pyk commented 7 years ago

oh ya tambahan, ternyata ada field mentions juga.

Jadi kalau ada request dengan fields mentions dan sentiments kita tampilin news yang is_analyzed=True aja

bobbypriam commented 7 years ago

@pyk Itu gapapa? Maksudnya berarti hasilnya beda kan, padahal embed mengesankan kita cuma nempelin informasi tambahan. Atau sekalian aja di /news kita cuma tampilin yang udah is_analyzed? Kayak gitu, atau alternatifnya lagi kita tambahin satu query param untuk filter based on is_analyzed.

Tapi kalo emang lebih setuju yang filternya ketika embed gapapa sih.

pyk commented 7 years ago

Atau sekalian aja di /news kita cuma tampilin yang udah is_analyzed?

Setuju 👍 , mending kita expose ke public yang udah di analyze Rojak aja. Jadi hasilnya gak beda juga tanpa/pake embed.

bobbypriam commented 7 years ago

Okay, saya bakal ubah besok :+1:

bobbypriam commented 7 years ago

@pyk Saya baru push lagi v0.3.0-beta.1, boleh dicoba dulu? 🙇

pyk commented 7 years ago

@bobbypriambodo looks good 🙇

Btw untuk generate ini http://rojak-api-spec-staging.surge.sh/ lewat folder mana ya?

pyk commented 7 years ago

oh iya, belum di merge ya 😅

bobbypriam commented 7 years ago

@pyk Yang surge dari lokal saya mas hehe, iya nanti kalo udah di-merge bisa generate kayak biasa lagi. Oke kalo gitu ini saya bump jadi v0.3.0 ya.

bobbypriam commented 7 years ago

Thanks mas!