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

CTP-3660 LTI source for Marks Transfer #65

Closed

Conversation

watson8
Copy link
Contributor

@watson8 watson8 commented Oct 1, 2024

Draft pull request for review as discussed. I did not yet increment the version number as I have a separate branch for each and I do not know which is likely to be merged first so thought it best to avoid conflict. There are similar branches for the other 3 mod types (Lesson, LTI, HVP) under https://github.com/ucl-isd/moodle-local_sitsgradepush/branches

Copy link
Collaborator

@aydevworks aydevworks left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done review.

classes/assessment/lti.php Outdated Show resolved Hide resolved
namespace local_sitsgradepush\submission;

/**
* Class for coursework plugin (mod_coursework) submission.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Class for LTI?

@aspark21
Copy link
Collaborator

Superseeded by #71

@aspark21 aspark21 closed this Oct 10, 2024
@aydevworks aydevworks deleted the CTP-3660-Add-support-for-lti-source-for-Marks-Transfer branch October 28, 2024 16:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants