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-3563 add mod_coursework support #64

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

@watson8 watson8 requested a review from aydevworks October 1, 2024 14:28
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.

Looks good.

classes/assessment/coursework.php Outdated Show resolved Hide resolved
classes/submission/coursework.php Outdated Show resolved Hide resolved
@watson8
Copy link
Contributor Author

watson8 commented Oct 7, 2024

Addressed comments in commit 5ad495c

@aspark21
Copy link
Collaborator

Superseeded by #71

@aspark21 aspark21 closed this Oct 10, 2024
@aydevworks aydevworks deleted the CTP-3563-add-support-for-coursework-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