-
Notifications
You must be signed in to change notification settings - Fork 40
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
Add endpoint to get all materials for a user #1644
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes, ok.
becba2e
to
141aa6b
Compare
Tests are blocked on #1646 |
/usermaterials/${id} returns a list of all the learning materials a user is connected to as a learner.
This is likely to be volatile for a while until we figure out exactly what belongs here.
Calculates the first offering for a materials based on the students association via IlmSession or Offering.
2051a2d
to
5e74799
Compare
Data fixtures were modified for testing usermaterials endpoint and these tests needed some touchups.
5e74799
to
326afec
Compare
@jrjohnson i don't see anything in the repo query function that pertains to checking the owning session/course for publication status. not sure if this an issue or not, but it may be. thoughts? |
@stopfstedt good catch. |
LMs connected to items in draft or scheduled state should not be displayed for learners attached to those courses / sessions.
@stopfstedt ready for review again. |
/usermaterials/${id} returns a list of all the learning materials a user
is connected to as a learner.
Fixes #1635