-
Notifications
You must be signed in to change notification settings - Fork 34
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
Sakai LTI compatibility. LTI module combined with app. #1428
Open
iturgeon
wants to merge
6
commits into
ucfopen:master
Choose a base branch
from
iturgeon:issue/lti-compatability-with-sakai
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Sakai LTI compatibility. LTI module combined with app. #1428
iturgeon
wants to merge
6
commits into
ucfopen:master
from
iturgeon:issue/lti-compatability-with-sakai
Commits on Dec 31, 2022
-
LTI overhaul to support Sakai 22.x
* Change var to const for js constants being injected into the page from the server * Allows the before_play_start event to determine if the lti content item picker should be displayed * Move picker display function into a common controller * Add an api method that’ll validate and sign a content item selection lti request * injects LTI_MESSAGE_TYPE and LTI_KEY js constants into the picker page template * LTILaunch gains a method to load the lti config that is associated with a lti key * Updated a few strings that specifically mention canvas * Moves all classes, tests, migrations, and configs from fuel/app/modules/lti into fuel/app * overhaul lti test provider to simplify * simplify lti launch picker code when sending content selection messages to lti/assignment * fixed issue with starting plays with referrer urls longer than 255 characters fixes ucfopen#1418 * updates oauth validation in fuel by ensuring the request uri is not masked by FuelPHP * Overhaul lit test provider a bit - mostly to simplify and use lessons * fix code sniff errors * add deps to docker image * attempt to fix yarn cache issue * Changes docker build process to ensure the built image has admin:make_paths_writable run to avoid having to run it every startup
Configuration menu - View commit details
-
Copy full SHA for 702747b - Browse repository at this point
Copy the full SHA 702747bView commit details
Commits on Jan 25, 2023
-
Configuration menu - View commit details
-
Copy full SHA for c705e91 - Browse repository at this point
Copy the full SHA c705e91View commit details
Commits on Feb 2, 2023
-
Configuration menu - View commit details
-
Copy full SHA for 3ed6072 - Browse repository at this point
Copy the full SHA 3ed6072View commit details -
Configuration menu - View commit details
-
Copy full SHA for e021f5a - Browse repository at this point
Copy the full SHA e021f5aView commit details
Commits on Feb 4, 2023
-
Configuration menu - View commit details
-
Copy full SHA for acbfbd7 - Browse repository at this point
Copy the full SHA acbfbd7View commit details -
Configuration menu - View commit details
-
Copy full SHA for 63384fe - Browse repository at this point
Copy the full SHA 63384feView commit details
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.