provide course grade in Gradebook REST webservice



Keyboard shortcuts  
Summarize the review outcomes (optional)


Warning: no files are visible, they have all been filtered.
Participant Role Time Spent Comments Latest Comment
Author 17m 7 Renamed field to courseGrade to make it clearer and added...
Reviewer - 100% reviewed 25m 4 This is the only place in the patch that I see the usage ...
Reviewer - Complete 6m    
Reviewer - Complete 4m 2 I'd update the comment - this method isn't actually doing...
Total   53m 13  


It is desirable to have course grade information there, too:
1) student course grade -> included in "/grades/students" feed
2) average course grade -> included in "/grades/gradebook" feed

Branches in review


Issues Raised From Comments

Key Summary State Assignee

General Comments

There are no general comments on this review.
/gradebook-service/.../shared/ Changed  
Open in IDE #permalink
/gradebook-service/.../gradebook/ Changed   8
Open in IDE #permalink
/grades-rest/src/.../entity/ Changed  
Open in IDE #permalink
/grades-rest/src/.../entity/ Changed   3
Open in IDE #permalink
/grades-rest/src/.../logic/ Changed   2
Open in IDE #permalink

Review updated: Reload | Ignore | Collapse

You cannot reload the review while writing a comment.

Create issue

Assign To Me

Log time against