moravianlibrary / kramerius

GNU General Public License v3.0
1 stars 1 forks source link

Title v response /children ma jiny typ #54

Closed ovyhlidal closed 8 years ago

ovyhlidal commented 8 years ago

Pokud poslu request na /children pro monograph dostanu v response title jako array. Pokud ovsem poslu stejny request pro soundrecording, tak je v response title jako string. Z logickeho uhlu pohledu by tento typ mel byt u obou requestu stejny.

Request pro monograph: http://kramerius.mzk.cz/search/api/v5.0/item/uuid:530719f5-ee95-4449-8ce7-12b0f4cadb22/children

Request pro soundrecording: http://kramerius.mzk.cz/search/api/v5.0/item/uuid:59e708b6-c462-4610-90c5-ac5ca030050a/children

Jenyk commented 8 years ago

To není otázka modelu dokumentu, ale bug krameria při vracení odpovědi v json. Viz https://github.com/ceskaexpedice/kramerius/issues/211

leiblix commented 8 years ago

Už je to opravené. Nasadil jsem aktuální verzi na test http://krameriustest.mzk.cz/search/api/v5.0/item/uuid:530719f5-ee95-4449-8ce7-12b0f4cadb22/children

Zítra to dáme na produkci.

leiblix commented 8 years ago

Domluvil jsem se s @ovyhlidal, že zatím to nasazení odložíme, rozbili bysme mu tím jeho testovací release (počkáme na další).

zabak commented 8 years ago

Nerozbije to náhodou i klienta pro Android?

leiblix commented 8 years ago

@zabak V pátek jsme to s @honza-rychtar zkoušeli a bylo to v pohodě.

ovyhlidal commented 8 years ago

Tak pokud to nerozbije klienta pro Android, tak bych osobne navrhnul to nasadit co nejdriv a ja hned vydam novou testovaci verzi iOS klienta, kde tohle bude upravene, at na to nemusime porad myslet. Co na to @zabak @MartinRumanek ?

leiblix commented 8 years ago

Honza zjistil, že v upstreamu nefunguje korektně volání reindexací přes remote api (starší verze je ok), bude vhodné to před nasazením vyřešit.

@Jenyk dopíšeš prosím stručně podrobnosti?

leiblix commented 8 years ago

Blokováno https://github.com/ceskaexpedice/kramerius/issues/324

leiblix commented 8 years ago

Vyřešeno aktualizací.