proarc / proarc-client

3 stars 0 forks source link

Svazek vícedílné monografie - chyby ve v. 4.1.0-VISK23-beta/v2.2.0-VISK23 (12.09.2023) #433

Closed ZdenkaSera closed 8 months ago

ZdenkaSera commented 10 months ago

Svazek vícedílné monografie lze založit plusem přímo pod vícedílnou monografií, ale nelza založit jako samosttaný objekt

image

ZdenkaSera commented 10 months ago

Když zakládám svazek vícedílné monografie pluskem přímo pod VD, neměla by se mi vícedílná monografie nabízet jako možný objekt k založení - VD pod VD není správně

image

ZdenkaSera commented 10 months ago

V importu nemohu nové skeny přiřadit ke svazku vícedílné monografie

image

Tady ještě doplním - pokud si v importech nechám zobrazit "vše" (tj,. neomezím model) a výběr nadřazeného objektu omezím názvem, zobrazí se mi i svazek vícedílné monografie. Takže chybu lze obejít, ale opravu to chce.

ZdenkaSera commented 10 months ago

@SykoraLukas @albertoh Tmuhle jsem dala prioritu, protože v téhle podobě nelze verzi použít. Předem díky!

SykoraLukas commented 10 months ago

Vezmu to postupně:

Založení samostatně opravíme. VD pod VD se řešilo zde https://github.com/proarc/proarc-client/issues/99#issuecomment-836288501 a je tam i zmíněno, jak toto vzniklo. Zároveň je tam řečeno, že se to rozhodne až vyjde nový standard. Takže jak to má být? @nezbedova, @ZdenkaSera , @katerinasirova, @erikamirova a @luckajirku?

Přiřazení stran opravíme.

Label k modelu je uloženen v cs.json a en.json, ty bude potřeba také přepsat při nasazení, ve vydané verzi to je v pořádku.

ZdenkaSera commented 10 months ago

V KNAV rozumíme i novému standardu tak, že VD pod VD nelze - řešili jsme s @kerschfilip a @SmejkalovaAnna.

luckajirku commented 10 months ago

Ty úpravy podle nového standardu se týkaly rozsahu popisu atd., na to se mělo počkat s novým standardem, ale VD pod VD neměla jít založit nikdy, to byl nesmysl od začátku.

albertoh commented 10 months ago

Opraveno

ZdenkaSera commented 10 months ago

Nenašla jsem zatím problém - @SykoraLukas @albertoh děkuji.

ZdenkaSera commented 9 months ago

Testováno v KNAV na posledních verzích 4.1.0-VISK23-beta/v2.2.0-VISK23 (12.09.2023)

Založení svazku VD jako samostatný "nový objekt" se zřejmě nedostalo do verze v kompletně opravené podobě. V seznamu modelů si můžu vybrat model.model:ndkMonographUnit, můžu vyhledat data v katalogu, ale pak končím s chybou:

image

Děláme něco špatně?

ZdenkaSera commented 9 months ago

Importované skeny svazku VD v KNAV nepřiřadím (4.0.7-DEV/v2.1.1 (24.07.2023)): image

SykoraLukas commented 9 months ago

Nad jakou verzi API a Klienta to prosim delas?

ZdenkaSera commented 9 months ago

@SykoraLukas Psala jsem výše - na posledních vydaných, tj. 4.0.7-DEV/v2.1.1 (24.07.2023); ta verze klienta je asi špatně pojmenovaná, tohle číslo ukazuje v aplikaci, Opravuji verze - 4.1.0-VISK23-beta/v2.2.0-VISK23 (12.09.2023)

SykoraLukas commented 9 months ago

Tohle ale nejsou poslední verze. Poslední je Klient: verze v2.2.0-VISK23 (31.08.2023) a Jádro: verze 4.1.0-VISK23-beta

Prosím o update na aktuální verze.

SykoraLukas commented 9 months ago

Na inovatice toto funguje

image
ZdenkaSera commented 9 months ago

@SykoraLukas Promiň, dělalm jsem paralelně něco na ostrém PA a opsala verze odtud - na testu máme 4.1.0-VISK23-beta/v2.2.0-VISK23 (12.09.2023). Jinak všechno výše platí

ZdenkaSera commented 9 months ago

Ano, na Inovatice to funguje, tam jsem to testovala dříve a issue označila k uzavření. Podle mého to jen nebylo zahrnuto do verze.

SykoraLukas commented 9 months ago

@albertoh mohl bys na to kouknout? Mě se zdá, že to ve verzi zahrnuté je. Nějaký další nápad, proč se jim to takto tváří?

SykoraLukas commented 9 months ago

Mě se to trochu jeví, jako že máte nasazenou minulou verzi. Tam to takhle bylo. A hlavně mi přijde divné, že nemáte přeložený ten model do čj ale máte kod.

ZdenkaSera commented 9 months ago

Tohle je v aplikaci, na nižší verzi to nevypadá ...

image

albertoh commented 9 months ago

Jsem to zbuildoval znovu. https://github.com/proarc/proarc-client/releases/tag/v2.2.1-VISK23

Zkuste to znovu stahnout a nasadit

ZdenkaSera commented 9 months ago

@albertoh Na https://proarc-master.inovatika.dev/ není ve seznamu k založení jako "nový objekt" ani v seznamu modelů k hledání NDK svazek vícedílné monografie. (Je k dispozici jen při zakládání nového objektu pluskem.

albertoh commented 9 months ago

Pridano

ZdenkaSera commented 9 months ago

@albertoh Díky, funguje.

ZdenkaSera commented 9 months ago

@albertoh

ZdenkaSera commented 9 months ago

@albertoh Tohle jsem zavřela omylem - potvrzené fungování se týkalo masteru. U nás na testu @PetrKotynek nasadil po vydání verzi v2.2.1-VISK23 uvedenou výše, ta se ale nelišila od té předchozí, snažil se o re-build. Svazek vícedílné monografie tedy můžeme založit pluskem, ale ne samostatně a ne vyhledávat, tam je stále nefunkční "model.model:ndkMonographUnit"

albertoh commented 9 months ago

v global.js musi byt mezi modely tohle:

'model:ndkmonographunit',

Pozor na velka a mala pismena

PetrKotynek commented 9 months ago

Změněno na proarc-test@proarc-dc jens test PK

ZdenkaSera commented 9 months ago

@albertoh Tak za tohle se moc omlouvám - v globals jsme to měli, to jsme kontrolovali, ale vloudilo se tam velké písmeno podle staršího vzoru, jak to psával tvůj předchůdce. Takže ještě lokálně přetestuju funkčnost, ale tohle bude zřejmě ok.