proarc / proarc-client

3 stars 0 forks source link

Chyba pri ukladani noveho uzivatele #319

Closed pkudela closed 1 year ago

pkudela commented 1 year ago

Pri refactoringu administrace uzivatelu jsem narazil na problem s ukladanim noveho uzivatele, zahlasi nize uvedeny error:

image

@albertoh muzes se na to prosim podivat? Dik

albertoh commented 1 year ago

Hlaska povida "Invalid username". Ted jsem zalozil uzivatel test4 bez problemu. @SykoraLukas jake jsou pravidla pro uzivatelska jmena?

SykoraLukas commented 1 year ago

Pravidla jsou:

ZdenkaSera commented 1 year ago

@albertoh přidá kontrolu, zda username odpovídá šabloně.

ZdenkaSera commented 1 year ago

Otestováno v KNAV - pokud dodržím výše popsaná pravidla, uživatele lze založit, jinak uživatelsky nepřívětivá chybová hláška. . Kontroly jsou zatím jen na povinnost polí.

albertoh commented 1 year ago

Pridana kontrola a hlaska

ZdenkaSera commented 1 year ago

@albertoh Alberto, vidím u každého pole nápovědu, to je super 👍 Ale při pokusu o uložení nesmyslných hodnot to pořád vyhazuje stejnou nevlídnou chybu. Co dělám špatně? Reloadem by to být nemělo ... Díky.

image

SykoraLukas commented 1 year ago

Opraveno, prosím o vyzkoušení.

ZdenkaSera commented 1 year ago

Uživatel je kontrolován na tvar, heslo na délku - za mne ok, děkuji.

luckajirku commented 1 year ago

Uživ. jméno je červené, když není OK. Ale heslo, i když nesplňuje podmínky, zůstává šedě - nemělo by se taky probarvit?

ZdenkaSera commented 1 year ago

Nechám rozhodnutí na ostatních, v tomhle případě se mi zdá zásadní funkční kontrola. Děkuji.

pkudela commented 1 year ago

Toto by melo byt opraveno, udelal jsem novou kontrolu poli, ted uz nevyskakuji dialogy, ale pole se oznaci a vygeneruje hlasku.

image

Jakmile bude nasazeno, oznacim k testovani.

pkudela commented 1 year ago

nasazeno

luckajirku commented 1 year ago

Btw ty fce převodu, updatu, uzakmknutí/odemknutí objektu nemají superadmin a admin přednastavené? Musí se to i u nich navolit?

ZdenkaSera commented 1 year ago

Tady ještě detail - mám chybu v uživateli, přesto se rozhodnu záznam uložit. Správně se neuloží, ale mohla by ta hláška být přátelštější ? Není prio. image

ZdenkaSera commented 1 year ago

Prosím o radu - do nastavení uživatelů můžu už jako admin - chceme to tak? Připadalo by mi správnější, kdyby tam mohl jen superadmin - ale možná mi někde zase unikla diskuse k tématu.

ZdenkaSera commented 1 year ago

Takže je to ještě jinak - admin má zřejmě právo jen zobrazit a editovat uživatele z vlastní organizace, založit nového uživatele můžu jen jako superadmin. Tak zatím ok, praxe ukáže.

ZdenkaSera commented 1 year ago

Zavírám dle domluvy ze schůzky 15. 6. 2023.