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

Implement removal of attested results #63

Open
dbosk opened this issue Feb 17, 2023 · 1 comment
Open

Implement removal of attested results #63

dbosk opened this issue Feb 17, 2023 · 1 comment

Comments

@dbosk
Copy link
Owner

dbosk commented Feb 17, 2023

This is needed if one wants to change a grade, plussning. Particularly if there is a bulk of them (in the UI it must be done per student, each step requiring a login).

@dbosk dbosk assigned dbosk and unassigned dbosk Jul 5, 2024
@dbosk
Copy link
Owner Author

dbosk commented Jul 5, 2024

@alba-kth För attesteringen, läs kapitel 9 (sidorna 44-48) och 13 (496-520) i PDF:en.

Om du kollar i innehållsförteckningen ser du att kapitel 13 (specifikt avsnitt 13.4, där attesteringen ska in), så går den från klarmarkering (13.4.3) till uppdatera klarmarkerat (13.4.4), ta bort klarmarkering (13.4.5) och ta bort utkast (13.4.6). Testerna är designade att köras i ordning så att de börjar med att lägga till ett resultat och sedan successivt ta bort det igen. Så det borde vara lämpligt att klämma in attestera och ta bort attestering mellan 13.4.3 och 13.4.4.

Sedan bör det hamna lite kod i den objektorienterade delen (kapitel 8 ) också i sinom tid. Typ en metod attest() och en grade_improvement(new_grade). Tänker mig att det blir lämpligt med grade_improvement för plussning så att folk inte råkar ändra datumet, utan bara ändrar betyget. (CSN vill att datumet inte ska ändras, jag har verifierat det med dem.)

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

No branches or pull requests

1 participant