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.

Sign up for GitHub

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

Jump to bottom

Time period mapping should not be required #19

Open
timhodson opened this issue Sep 2, 2016 · 0 comments
Open

Time period mapping should not be required #19

timhodson opened this issue Sep 2, 2016 · 0 comments
Labels
enhancement LTI Activity Plugin

Comments

@timhodson
Copy link
Member

timhodson commented Sep 2, 2016 •
edited
Loading

in aspirelists/lib.php, when we have set knowledge_grouping_code to be the course code, we should not expect to then look for a time period mapping object.

Make it optional.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement LTI Activity Plugin
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@timhodson