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

API endpoint: IDs objektu #593

Open
JanMeritus opened this issue Jul 29, 2021 · 3 comments
Open

API endpoint: IDs objektu #593

JanMeritus opened this issue Jul 29, 2021 · 3 comments
Assignees
Labels
Milestone

Comments

@JanMeritus
Copy link
Contributor

Lze se dotazovat na denni IDs sklizni (den -> IDs sklizni + IDks jejich kolekci (pokud je maji) ) #468

Kazda sklizen ma sve ID a lze jeji JSON servirovat po dotazu, obsahuje taky IDks sklizni.

Kazda interni kolekce a kazda externi kolekce ma sve ID a lze se cez toto (IDk) na ne dotazovat via API. (vazba #592)

Dynamicke kolekce (typ M1-M12, ArchiveIt, Test, OneShot) sve ID prozatim postradaji. (vazba #402)

@Fasand
Copy link
Contributor

Fasand commented Sep 1, 2021

Dávám sem návrh jak by endpointy mohly vypadat

Denní / By Date

/seeder/api/harvest/date/<dd-mm-YYYY> : napadá mě formát { harvestId : [internalCollectionIds] } jestli může být.
Např. pokud mám sklizně 1,2 a kolekce 11,12: {1: [], 2: [11,12]}

Harvest JSON

/seeder/api/harvest/<id harvest>/json : vlastně celá #402

Tématické kolekce

/seeder/api/tc_internal/<id internal tc>/json : může vrátit JSON kolekce tak, jak by se zobrazil v Harvest JSONu
/seeder/api/tc_external/<id external tc> : podobně pro externí kolekce, ale co by v tomto případě měla vlastně vrátit? Prostě všechna svá pole + třeba IDs interních kolekcí? Nebo rovnou ten JSON co vrátí endpoint pro interní kolekce?

Udělal bych to přes rest framework, takže zatím může být přístup pro přihlášené a potom to změním třeba na token, zároveň to API bude částečně "před-generované", takže bude mít automaticky List a Detail pro ID.
Jde mi tedy trochu o sémantiku: u Harvest JSONu mi dává smysl přidat /json, protože to není jenom výpis polí ale je to něco custom, každopádně u externí kolekce můžu prakticky jen vypsat ta pole, takže mi tam to /json nesedí.

@JanMeritus
Copy link
Contributor Author

JanMeritus commented Oct 22, 2021

@Fasand

Denni / By Date

  • vrati spet datum, pocet sklizni ID jedntolivych sklizni a ich aktualni stav

{ date: dd-mm-YYYY,
harvestCount: integer,
harvests:
[ harvestID : harvestStatus,
harvestID : harvestStatus ]}

Harvest JSON

ok

Kolekce

ano, nazev ale ne tc_internal, tc_external ale:

  • collections/internal//json
  • collections/external/ (vrati pole a ID internich kolekci)
  • collections/internal collections/external (vrati IDy externich i internich kolekci)

@mariehaskovcova mariehaskovcova added this to the 1.0.11 milestone Apr 12, 2024
@mariehaskovcova
Copy link
Contributor

příležitostně prosím zhodnoť @dragounv, díky

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

No branches or pull requests

3 participants