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

Mazanie InforequestEmail #311

Closed
martinmacko47 opened this issue Oct 22, 2020 · 1 comment
Closed

Mazanie InforequestEmail #311

martinmacko47 opened this issue Oct 22, 2020 · 1 comment

Comments

@martinmacko47
Copy link
Owner

martinmacko47 commented Oct 22, 2020

Analogicky ako sme v #275 a #276 mazali akcie resp. branche, upravit aj mazanie InforequestEmail instancii. Zakazat hromadne mazanie a povolit iba jednotlive mazanie instancii.

Zmazat sa budu dat iba tie InforequestEmail instancie, ktore odkazuju na email Message, ktory nie je pouzity v ziadnej action danej infoziadosti. Teda, ze vzdy musi byt splneny invariant, ze vsetky Message pouzite v akciach infoziadosti, musia byt nalinkovane na infoziadost aj cez InforequestEmail

Edit: Alternativne, ak by sa dalo, tak mazanie instancie nezakazat tak, ze sa skryje "delete" button. Ale namiesto toho na obrazovke, kde admin potvrdzuje mazanie instancie, zobrazit informaciu, ze instanciu nejde zmazat, spolu s dovodom preco, a az tam skryt button na potvrdenie mazania. A tiez samozrejme zabespecit, aby instanciu neslo zmazat, ani ked admin "klikne" na ten nezobrazeny button rucnym poslanim HTTP requestu.

Ak by sa toto podarilo, tak potom by sme mohli povolit aj hromadne mazanie. Ak by admin pri hromadnom mazani oznacil aj instanciu, ktoru nejde zmazat, tak na obrazovke potvrdzujucej mazanie by sa rovnako zobrazila chyba a button na zmazanie by nebol zobrazeny. Ak by admin oznacil iba instancie, ktore sa daju zmazat, tak hromadne mazanie by fungovalo standardne.

Ak sa to podari, tak potom v dalsich taskoch mazanie ostatnych modelov upravime rocnako.

Edit 2: Vyzera, ze hromadne mazanie so zakazanim mazania konkretnych instancii nevieme tak jednoducho spravit, bez vyrazneho prekodenia hromadneho mazania. Takze hromadne mazanie nechajme zakazane a povolme iba mazanie instancii po jednej a spravme to analogicky ako v #275 a #276. Akurat by sa zislo na detaile intancie okrem skrytia delete buttonu napisat niekam aj dovod, preco tuto instanciu nejde zmazat.

@martinmacko47 martinmacko47 added this to the 2020 milestone Oct 22, 2020
@martinmacko47 martinmacko47 changed the title Admin: Zmazanie InforequestEmail Admin: Mazanie InforequestEmail Oct 22, 2020
@martinmacko47 martinmacko47 changed the title Admin: Mazanie InforequestEmail Mazanie InforequestEmail Nov 4, 2020
viliambalaz added a commit that referenced this issue Nov 22, 2020
@martinmacko47
Copy link
Owner Author

Scope tohto tasku sa presunul do #334

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants