Skip to content
This repository has been archived by the owner on Jan 26, 2022. It is now read-only.

Releasing grades #15

Open
rtholmes opened this issue Oct 20, 2016 · 4 comments
Open

Releasing grades #15

rtholmes opened this issue Oct 20, 2016 · 4 comments

Comments

@rtholmes
Copy link
Contributor

It would be great to be able to release the D1 grades so the students can see them; I have set gradesReleased: true in deliverables.json but that doesn't seem to do it. Do I have to do anything else?

@mksarge
Copy link
Contributor

mksarge commented Oct 20, 2016

Working on it today!

@rtholmes
Copy link
Contributor Author

Deployed, looks great! Just checking: is it only the client that decided to render the grade if released==true (aka do the clients get their grades if they look in the developer console, or is the data not sent if released==false)?

Also, when the students view their comments could the button say 'Close' instead of 'Cancel'.

@mksarge
Copy link
Contributor

mksarge commented Oct 20, 2016

@rtholmes Ah good point, yes I only blocked the grade from rendering on the client, but the grades are still being received from the server. I should block the grade from being sent on the server side, if gradesReleased=false.

@rtholmes
Copy link
Contributor Author

For today it isn't urgent, because we want them to see their grades anyways, but in future it would be handy. Right now I think #13 is more important since it is probably easy for you but not so easy for me.

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

No branches or pull requests

2 participants