provide course grade in Gradebook REST webservice

Activity

SAKTRUNK-81 13

Keyboard shortcuts  
Summarize the review outcomes (optional)
 
#permalink

Details

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  
#permalink

Objectives

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

#permalink

Issues Raised From Comments

Key Summary State Assignee
#permalink

General Comments

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

Review updated: Reload | Ignore | Collapse

You cannot reload the review while writing a comment.

Create issue

X
Assign To Me

Log time against