Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Kickoff GraphQL servera pre Otvorené API #2

Open
michalkvasnicak opened this issue Aug 18, 2016 · 12 comments
Open

Kickoff GraphQL servera pre Otvorené API #2

michalkvasnicak opened this issue Aug 18, 2016 · 12 comments

Comments

@michalkvasnicak
Copy link

Rýchly kick off GraphQL servera https://github.com/michalkvasnicak/open-api-graphql

Je to robené na rýchlo tak som previedol len CRZ avšak pozmenil som schému response aby mala tvar vhodnejší pre GraphQL.

Vytvárajte PR pre ďalšie schémy :) prípadne pre opravy.

@jsuchal
Copy link
Member

jsuchal commented Aug 19, 2016

@michalkvasnicak sorry za neznalost, ale aka je vyhoda v tomto?

@michalkvasnicak
Copy link
Author

Ani by som nepovedal, že je to nejaká výhoda, skôr len enhancement ak by niekto chcel skladať nejakú appku a nechcel mať v response všetky dáta ale len to čo potrebuje.

@jsuchal
Copy link
Member

jsuchal commented Aug 19, 2016

ok chapem, inak aktivite sa medze nekladu, ale momentalne by sme skor prijali nejake tie appky a ziskavanie dalsich dat :)

@michalkvasnicak
Copy link
Author

Rozumiem, ináč je niekde dostupný endpoint na získanie zoznamu zmlúv? Pretože som sa dopátral len k detailu. Možno som to prehliadol v noci.

@jsuchal
Copy link
Member

jsuchal commented Aug 19, 2016

@michalkvasnicak zoznam zmluv? je tam api na synchronizaciu vid https://ekosystem.slovensko.digital/otvorene-api#získanie-zmenených-zmlúv-od-daného-času to by nestacilo?

Lebo zmluv je vela celkom

@michalkvasnicak
Copy link
Author

Ide to, že takto musím vytvoriť vlastný backend k tomu aby som tieto zmluvy uložil, tento graphql server slúži len ako jednoduchý wrapper okolo existujúceho API.

Momentálne nie je možnosť prechádzať si zoznamom zmlúv. Je nejaký dôvod prečo na to nie je API endpoint? Klasická paginácia page + limit?

@jsuchal
Copy link
Member

jsuchal commented Aug 19, 2016

@michalkvasnicak klasicka paginatia page + limit ma velky problem tu http://use-the-index-luke.com/no-offset

  • isli sme na uvod minimalisticky. Ak by som pridal paginate tak take api je sice pekne ale realne ho nikto nebude pouzivat lebo by videl asi nieco taketo.

image

Co sice na prvy pohlad vyzera rozumne, ale prakticke pouzitie je uplne nulove, kedze tych zmluv je cez milion a nikto to takto nebude realne robit. Resp. budem cakat kym ma niekto presvedci o opaku. Skor ma zmysel search api a tam radsej pockam na aplikaciu.

Cize pokial nemame poziadavku na toto z realnej aplikacie, tak to je IMHO predcasne.

@michalkvasnicak
Copy link
Author

michalkvasnicak commented Aug 19, 2016

Rozumiem. Ide o to, že na to aby som mohol použiť detail zmluvy, tak musím vedieť jej ID. Viem momentálne nejako zistiť toto ID? Alebo je to na komunite aby vzala dataset a spravila nad ním vlastné API?

@jsuchal
Copy link
Member

jsuchal commented Aug 19, 2016

Cez synchronizacne API sa dostanes k vsetkym ID aj obsahom.

Ale vratme sa este raz spat, to ID asi nechces len nejake nahodne. Co ma splnat ta zmluva, ktorej ID ta zaujima? Na toto musi by realisticky scenar. Ja ho nevidim.

Samozrejme radi dorobime API, len to musi mat zmysel.

@msimulcik
Copy link

@michalkvasnicak chvalim iniciativu s GraphQL. Pohol si sa s tym niekam dalej od augusta?

@jsuchal na to co potrebuje @michalkvasnicak by bolo treba pridat filtrovanie, sorting a cursor based pagination, aby to bolo pouzitelne. Tam potom ale moze byt problem ustrazit kombinacie vsetkych filtrov a sortingu, tak aby queries boli performant. Asi naozaj bude treba use case, aby sa to oplatilo robit.

Kazdopadne s tymto rad pomozem, keby bolo treba.

@michalkvasnicak
Copy link
Author

@msimulcik nemal som čas na to odvtedy moc + chcelo by to preniesť do nového stacku (napríklad, ktorý používam v práci) kde je generovanie schém jednoduchšie atď.

@jsuchal
Copy link
Member

jsuchal commented Nov 17, 2016

Hlavne stále nemáme reálny usecase na toto. Až príde tak budem prvý kričať
že toto spravme.

On Thu, 3 Nov 2016, 10:19 Michal Kvasničák, notifications@github.com
wrote:

@msimulcik https://github.com/msimulcik nemal som čas na to odvtedy moc

  • chcelo by to preniesť do nového stacku (napríklad, ktorý používam v
    práci) kde je generovanie schém jednoduchšie atď.


You are receiving this because you were mentioned.

Reply to this email directly, view it on GitHub
#2 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABm5sacBz7-NZ0n9sbXJBA5EwIixcIzks5q6acKgaJpZM4Jn_Fs
.

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

No branches or pull requests

3 participants