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

Roster update - search users with filter status Active #649

Open
stemar opened this issue Jul 14, 2020 · 0 comments
Open

Roster update - search users with filter status Active #649

stemar opened this issue Jul 14, 2020 · 0 comments
Labels
v3.4.6 Suggested updates for v3.4.6

Comments

@stemar
Copy link
Contributor

stemar commented Jul 14, 2020

I do not believe that you or the LTI lib you are using are filtering out users by status. Based on the approach you are using, you should only be using members with a membership status of Active https://www.imsglobal.org/spec/lti-nrps/v2p0#membership-status

this also applies to removeRemainingUsersFromIpeerRoster and addRemainingUsersInIpeerRoster

As an alternative, (if I remember correctly) ipeer can set users to an inactive enrollment status so you could do as you currently do but set the user's enrollment status to active/inactive instead (ex in addUserToCourse)

Originally posted by @andrew-gardener in https://github.com/_render_node/MDE3OlB1bGxSZXF1ZXN0UmV2aWV3NDExMTM4MDY5/pull_request_reviews/more_threads

@stemar stemar added the v3.4.6 Suggested updates for v3.4.6 label Jul 14, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
v3.4.6 Suggested updates for v3.4.6
Projects
None yet
Development

No branches or pull requests

1 participant