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

Dokud není žádost odeslaná, pak by měl mít možnost akce nad dokumentem i jiný uživatel #120

Closed
CIZKOVAL opened this issue Oct 20, 2021 · 16 comments
Assignees
Labels
bug Something isn't working c::Functions Cast - Funkce Done p:DEV Prostredi - Vyvoj p::Middle question Further information is requested
Milestone

Comments

@CIZKOVAL
Copy link
Collaborator

Dobrý den,
pokud uživatel 1 (např. Knihovna) zadá návrh na vyřazení, ale neodešle jej, pak další uživatel 2 (např. Nakladatel) nemůže podat návrh na vyřazení (u dokumentu je příznak "Záznam má návrh: VVS").

Knihovna_neodeslane_zadosti

Nakladatel_nema_moznost_vyradit

@CIZKOVAL CIZKOVAL added p:TEST Prostredi - Test c::Functions Cast - Funkce p::Middle labels Oct 20, 2021
@CIZKOVAL CIZKOVAL mentioned this issue Oct 20, 2021
18 tasks
@JanMeritus
Copy link
Contributor

@CIZKOVAL @pavel-stastny dle domluvy je zelana funkcionalita ze uzivatele muzu posilat podnety na dilo, ktere je jiz v procesu z podnetu jineho uzivatele. Prvne se vyridi prvni podnet (muze byt neopravneny), nasledne dalsi podnet (ten jiz muze byt opravneny). Dilo nesmi byt blokovano (locknuto) z duvodu podnetu jineho uzivatele. Prosim o potvrzeni @Cypzirschova

@JanMeritus JanMeritus assigned pavel-stastny and unassigned pkudela Oct 21, 2021
@Cypzirschova
Copy link
Collaborator

Dílo nesmí být blokované pro jiné uživatele.

@CIZKOVAL
Copy link
Collaborator Author

Dobrý den,
doplňuji jak by to mělo fungovat a poprosím o potvrzení paní @Cypzirschova, zda souhlasí s popisem očekávaného chování:

žádost na změnu 1
uživatel A
uživatel B

Očekávané chování:

uživatel A - původní stav 0 - zadá žádost na změnu 1 --> okamžitě se změní stav 1
dokud není žádost schválená kurátorem, pak:
uživatel B - vidí původní stav 0 - může zažádat o změnu 1 -- změní se stav 1

Nyní se aplikace chová následovně:

uživatel A - původní stav 0 - zadá žádost na změnu 1 --> zůstává původní stav - ten se změní buď po refresh stránky nebo po odeslání žádosti
uživatel B - vidí stav 1 - nemůže zažádat o změnu

@Cypzirschova
Copy link
Collaborator

Jen mi není jasný pojem "zadá žádost na změnu 1 --> okamžitě se změní stav 1" - stav by se měl měnit až po rozhodnutí kurátora.

Jinak to je, tak jak píše paní Čížková. Díla nesmí být blokovaná pro jiné uživatele, jak je to v současné době nastavené v aplikaci.

@CIZKOVAL
Copy link
Collaborator Author

Omlouvám se, stavem je myšlen interní stav, tj. akce (návrh na zařazení/vyřazení)...

@Cypzirschova
Copy link
Collaborator

Tak takhle je to správně, jak je výše uvedeno.

@CIZKOVAL
Copy link
Collaborator Author

Upravuji tedy takto:

interní stav 0 = akce
stav 0
uživatel A
uživatel B

Očekávané chování:

uživatel A - původní stav 0 - interní stav 0 - podání návrhu --> stav 0, interní stav 1
dokud není žádost schválená kurátorem, pak:
uživatel B - původní stav 0 - interní stav 0 - může podat návrh --> stav 0, interní stav 1

Pozn. poté, co kurátor schválí návrh, tak se změní i stav z 0 -->1

Nyní se aplikace chová následovně:

uživatel A - původní stav 0 - interní stav 0 - podání návrhu --> stav 0, interní stav 0 (ten se změní buď po refresh stránky nebo po odeslání žádosti)
uživatel B - původní stav 0 - interní stav 1 - nemůže zažádat o změnu

@pavel-stastny
Copy link
Collaborator

Změněno v novém layoutu

@CIZKOVAL
Copy link
Collaborator Author

Dobrý den,
proces funguje správně v tomto ohledu:
Pokud uživatel A zadá žádost (návrh, tj. interní stav) na díle 1, pak se pro něj možnost další akce nad tímto dílem zablokuje.
Uživatel B může až do okamžiku zpracování žádosti kurátorem zadávat žádosti na díle 1.

Poznámka:

uživatel A (Knihovna) zadal návrh VN a poté se pro něj akce zablokuje, ale není zde zobrazen interní stav (konkrétně VN) - to by asi mělo být, to je ta změna interního stavu na 1

uživatel B (např. Lucie Čížková) může provádět akci nad dílem až do okamžiku zpracování žádosti. Tento uživatel interní stav nevidí, tj. OK

uživatel C (Kurátor) vidí interní stav (konkrétně VN) a může změnit stav na N-vyřazeno, toto je podle mě OK.

Poprosim pani @Cypzirschova o potvrzeni vyse uvedeneho.

knihovna_akce

CIZKOVAL_VN

kurator_VN

@CIZKOVAL CIZKOVAL removed the ToTests label Nov 30, 2021
@pavel-stastny
Copy link
Collaborator

@CIZKOVAL Jenom poznamka. NZN, VN, VNL a VNZ nejsou reprezentovany v aplikaci jako interni stavy ale vazby dokumentu na zadost. Stav nebo interni stav je vzdy exkluzivni. Tedy kdyz je A jiz nemuze byt PA, N, X atd... u vazeb tomu tak neni. Dokument muze mit vice vazeb na ruzne zadosti. (Jeden dokument muze byt v zadosti VN, VNZ i VNL a pokud je ve stavu PA tak i NZN)

@CIZKOVAL
Copy link
Collaborator Author

CIZKOVAL commented Dec 1, 2021

@pavel-stastny Dobrý den, tomu rozumím, že u 1ho dokumentu může být několik žádostí a mohou se lišit. Jde mi o to, že když já jako uživatel podám např. návrh VN, tak se pro mě už další akce nad tímto dílem zablokuje - šlo mi o to, jestli nezobrazit ten interní stav i pro tohoto uzivatele (ktery uz nemuze provest zmenu)

@pavel-stastny pavel-stastny added the question Further information is requested label Dec 2, 2021
@Cypzirschova
Copy link
Collaborator

Souhlasím, že by sám navrhovatel o vyřazení/zařazení měl vidět, proč už dál nemůže s dílem pracovat, když se jen jemu konkrétní dílo zablokuje. Viz komentář níže (úplně dole pracovní stavy).
#66 (comment)

@pavel-stastny
Copy link
Collaborator

@Cypzirschova Ok, můžu přidat informaci o žádosti i pro uživatele.

@Cypzirschova
Copy link
Collaborator

Jestli by to šlo, tak bych to dala jenom toho konkrétnímu uživateli (po přihlášení), který žádost podal.

@pavel-stastny pavel-stastny removed their assignment Dec 7, 2021
@CIZKOVAL CIZKOVAL removed the Done label Jan 5, 2022
@smallcubes smallcubes added the bug Something isn't working label Jan 25, 2022
@pavel-stastny
Copy link
Collaborator

@Cypzirschova @CIZKOVAL Uživatel má nyní informaci o tom, že má dokument v žádosti. Nasazeno na DEVu

@pavel-stastny pavel-stastny added Done p:DEV Prostredi - Vyvoj and removed p:TEST Prostredi - Test labels Feb 18, 2022
@CIZKOVAL
Copy link
Collaborator Author

CIZKOVAL commented Mar 4, 2022

Retest OK

VN_zadost

@CIZKOVAL CIZKOVAL closed this as completed Mar 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working c::Functions Cast - Funkce Done p:DEV Prostredi - Vyvoj p::Middle question Further information is requested
Projects
None yet
Development

No branches or pull requests

7 participants