search

water / mainline

Core web service
GNU Affero General Public License v3.0
9 stars 0 forks source link

Assistant Submission#show/edit #162

Open jesjos opened 12 years ago

jesjos commented 12 years ago

Jag har hittills använt Submissions#show för reviewing för assistenten. Nu när jag väl har lyckats få en review att fungera så inser jag att man hantera hur den vyn ska se ut när man väl har reviewat, labben är accepted eller rejected. Ska man kunna ändra informationen? Just nu tror jag state_machine hindrar transition direkt från rejected -> accepted Om man inte ska kunna ändra informationen längre så kanske man borde dela upp vyn i Submissions#show (går ej att ändra) och Submissions#edit (går att ändra).

Har folk något tips?

anjonas commented 12 years ago

Det vore nog bra om state inte kan ändras av en assistant när den är rättad, dvs state==accepted || state==rejected. När en student sen har gjort en till submissions och state==pending, så kan en assistant ändra state igen. Show kan användas för när state inte kan ändras och edit när den kan ändras.

Tarrasch commented 12 years ago

Jag vet inte riktigt om jag håller med dig @anjonas, har du tänkt på användarperspektivsidan om men assistent råkar missklicka bara? Känns som att en handledare måste kunna ångra, precist som Sven-Arne hela tiden sagt :p, det vet jag till och med Fire kan!

Fast rent tekniskt vi kanske bör hålla det på det sätt som du beskriver jonas, bara man kan ändra som handledare. :)