proarc / proarc-client

3 stars 0 forks source link

Hlášky, upozornění, varování atd. v novém klientovi #171

Open luckajirku opened 2 years ago

luckajirku commented 2 years ago
  1. Zřejmě neproběhla kontrola, zda jsou všechna upozornění a varování ze starého převedena i do nového klienta. Bylo by dobré to projet podle jádra - na to, co chybí, by neměli přicházet náhodně až uživatelé.

  2. Nebylo by od věci, kdyby proběhla i revize hlášek v novém klientovi - teď je to jak na perimetru - člověk neví, kdy/na jak dlouho, jak moc výrazná a na kterém místě obrazovky ta která hláška vyskočí - a nemusí si jí ani všimnout. Bylo by fajn mít v tom nějaký systém - jak mají vypadat (velikost, barva...) a chovat se různé typy hlášek - upozornění / varování / oznámení, že akce proběhla v pořádku... Taky kde na obrazovce se má zobrazovat (dole/střed/nahoře) a jestli si má vynucovat potvrzení (ano/ne nebo že to beru na vědomí), nebo má jen probliknout a zmizet...

ZdenkaSera commented 2 years ago

Právě jsem se chystala založit issue na odlišnost chybové hlášky mezi jádrem a klientem, založím nové issue a sem dám jen referenci.

ZdenkaSera commented 2 years ago

Hláška - export archivu neúplného objektu - #172

albertoh commented 2 years ago

Postupne prochazim/nachazim mista kde nejsou vychytane chyby. Vetsinou to nezobrazuje nic. Nekdy zustava tocici kolecko.

SmejkalovaAnna commented 2 years ago

Prosím, o změnu hlášky: pokud změním případně u NDK Periodika nějaká metadata a mám už pod tím Ročník a Číslo, tak mi vyběhne hláška: obrazek

albertoh commented 2 years ago

Misto "Nevalidní data" je "Upozornění"

SmejkalovaAnna commented 2 years ago

Díky, je tam ale "Varování" :) Ale je to OK, nechte to tak :)

ZdenkaSera commented 2 years ago

@albertoh Prosím opravit hlášku při ukládání nadřazeného objektu s nově připojenými skeny image

albertoh commented 2 years ago

Opraveno

ZdenkaSera commented 2 years ago

Export objektu - Prosím "Model" místo Moodel image

albertoh commented 2 years ago

@SykoraLukas to prichazi z jadra

ZdenkaSera commented 2 years ago

@albertoh Oprava v "Ukládání" (z 24.6.) je ok, díky.

ZdenkaSera commented 2 years ago

Pokud selže import archivu nebo k4, objeví se dvojitá hláška o chybě - viz níže - ale bez doplňující informace o důvodech. Lze tohle vylepšit?

image

ZdenkaSera commented 2 years ago

Chyba při přidělení urn:nbn vícedílné STT (ta chyba v ní opravdu je, tady jde jen o formát toho hlášení)

image

Doplňuji - problém je zřejmě i v samotném hlášení, založila jsem issue #218

SykoraLukas commented 2 years ago

Chyba při přidělení urn:nbn vícedílné STT (ta chyba v ní opravdu je, tady jde jen o formát toho hlášení)

image

Doplňuji - problém je zřejmě i v samotném hlášení, založila jsem issue #218

Tady to je chyba, co chodí přímo z resolveru NKCZ. S tím se nedá nic dělat. Kromě této chyby však API Proarcu posílá na klienta i jiné hlášky, což se změnou na tabulkové zobrazení vyřeší.

ZdenkaSera commented 2 years ago

Tady zatím odebírám label chyba, než sem zase něco přidáme.

erikamirova commented 1 year ago

V MZK na nás v případě např. exportní chyby vyskakují useknuté hlášky s podivným červeným rámečkem. Ve správě dávek je to ok.

image
ZdenkaSera commented 1 year ago

ečlánek nejde trvale smazat, pokud má nadřazený objekt . to je ok, ale hlášku prosíme upravit - NENÍ PRIO

image

luckajirku commented 1 year ago

Někam nám zmizela hláška Opouštíte formulář bez uložení... (https://github.com/proarc/proarc-client/issues/33)

pkudela commented 1 year ago

error dialog opraven

image
pkudela commented 1 year ago

uprava error dialogu nasazeno, prosim otestujte pro eclanek

ZdenkaSera commented 1 year ago

@pkudela Petře, tohle asi není k testu, při pokusu o trvalý výmaz připojeného e-článku to pořád hlásí totéž

pkudela commented 1 year ago

@ZdenkaSera ja pochopil, ze ti slo o fomat te hlasky, design, jak to bylo rozpadle. Jestli o funkci, tak prosim @albertoh mrkni se na to.

ZdenkaSera commented 1 year ago

@pkudela @albertoh Tak to se moc omlouvám, hlavně jsme chtěla, aby to mluvilo česky :-). Tohle ale počká klidně na další release,

pkudela commented 1 year ago

@ZdenkaSera takze Ti jde o text, to ze zahlasi chybu je v poradku? Cili tam chces hlasku ostraneni se nezdarilo, nebo neco podobneho?

ZdenkaSera commented 1 year ago

@pkudela Prosím zatím o text "Článek s nadřazeným objektem nelze smazat" Následně to ale asi budeme chtít povolit, zdá se, že to v přechozích verzích bylo možné mazat bez omezení. Interně ještě probereme. Děkuji!

pkudela commented 1 year ago

@ZdenkaSera opraveno, nasazeno na server.

katerinasirova commented 1 year ago

Ztratila se hláška při uložení objektu s nevyplněným M polem (v mém případě indexem strany) - a to jak při importu dat, tak v úložišti. Nic to nehlásí jak na inovatice, tak v naší instalaci.

SmejkalovaAnna commented 1 year ago

Pravda, je to tak, Popis Index zůstane červený, ale klidně mě to nechá uložit bez hlášky. V jádře mě to klidně nechalo uložit dávku bez jednoho vyplněného indexu a nic to nehlásilo..... ale to si nejsem jistá, jestli nebylo i předtím, ale mělo by to hlásit v každém případě, že ano? @katerinasirova

ZdenkaSera commented 1 year ago

@pkudela Design hlášky na Inovatice ok, díky. image

ZdenkaSera commented 1 year ago

@albertoh @pkudela odebírám "k testu" vzhledem k chybě od @katerinasirova výše.

kerschfilip commented 1 year ago

Při exportování skenů a původních skenů se ve správě dávek ukazuje divný profil: image

a taky prosím opravit hlášku, která se ukazuje, když smažu nějaké strany: image

testováno na v2.1.0 (14.06.2023)

pkudela commented 1 year ago

@kerschfilip ahoj, u toho prvniho i druheho chybi lokalizce / preklad, muzes mi prosim napsat, jak ma byt prelozen ten profil pro tento konkretni pripad? Dam to do lokalizace.

kerschfilip commented 1 year ago

@pkudela ve starém rozhraní u těch profilů ukazuje v případě exportu skenů i původních skenů Export - Datastream. Kdyby šlo rozlišit Export - Datastream Raw pro původní skeny a Export - Datastream Full pro export skenů, bylo by to super.

katerinasirova commented 1 year ago

Pravda, je to tak, Popis Index zůstane červený, ale klidně mě to nechá uložit bez hlášky. V jádře mě to klidně nechalo uložit dávku bez jednoho vyplněného indexu a nic to nehlásilo..... ale to si nejsem jistá, jestli nebylo i předtím, ale mělo by to hlásit v každém případě, že ano? @katerinasirova

Rozhodně by to mělo hlásit v každém případě.

ZdenkaSera commented 1 year ago

Přidávám sem prio 1 kvůli poslední diskutované chybě.

ZdenkaSera commented 1 year ago

@katerinasirova @SmejkalovaAnna Nepovedlo se mi přijít na to, kde je problém - testuju to blbě? Prosím o pomoc, jestli tam fakt je chyba. Děkuji.

Testovala jsem na Inovatice, stránky jsem připojovala k NDK mono. V importu jsem měla dávku, kde jsem nechala 2 strany bez indexu

image

image

SmejkalovaAnna commented 1 year ago

@ZdenkaSera Zdenko a co ti přijde jako chyba? Mě to přijde v pořádku.... hlavně, že to hlásí a neuloží se to, já v tom nevidím problém... že se to chová trochu jinak, to je asi daný tím druhem klienta ne? Nebo, co se ti na tom nezdá?

ZdenkaSera commented 1 year ago

@SmejkalovaAnna Aničko, mně se to zdá ok, ale ty s @katerinasirova jste řešily chybu - viz výše. proto se ptám, jestli je tam chyba, která mi uniká a testuju něco jiného, než jste měly na mysli. Visí nám to tu jako zásadní chyba před vydáním opravy k poslednímu releasu. Díky.

SmejkalovaAnna commented 1 year ago

Vypadá to už opravené.... chovalo se to právě předtím jinak. Teď to vypadá za mě ok.

SmejkalovaAnna commented 1 year ago

@ZdenkaSera @pkudela Tak zpět. při ukládání dávky je to v pořádku, ale v okně Editace (dřív vazby), když vymažu index, tak mě to nechá uložit bez hlášky, jen zůstane nadpis okna INDEX červený.... mělo by to při ručním přeuložení hodit nějakou hlášku, že to není OK.

ZdenkaSera commented 1 year ago

@albertoh @pkudela Prosím, mohli byste na tohle kouknout? Vázne na tom verze :-( Děkuji.

albertoh commented 1 year ago

To je na me. Podivam se, ale ve stredu

albertoh commented 1 year ago

Opraveno

SmejkalovaAnna commented 1 year ago

@albertoh @ZdenkaSera Otestováno, funguje.

katerinasirova commented 1 year ago

Za nás OK, otestováno a funguje.

ZdenkaSera commented 1 year ago

Moc díky všem, dávám sem návrh na uzavření.

ZdenkaSera commented 1 year ago

Tady odebírám všechny štítky a nechávám otevřené pro případné další opravy hlášek.