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

2021 07 13 rebase #33

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

Conversation

Dave-B
Copy link
Contributor

@Dave-B Dave-B commented Jul 27, 2021

Rebased onto 1.2.7 master.

Main changes:

Changes some hard-coded assignment fields into options.
Adds options for more assignment fields (including submission plugins).
Adds option section for course fields (currently only idnumber is added).
Adds option section for modifying how the options are displayed (currently only splitting first/last name into separate columns is added).
Adds a placeholder() function to help debugging.

Fixes #7.

David Balch added 8 commits July 27, 2021 16:38
"Username" is more properly the login username, which may be useful to
include as an option in the report.
Moves the hard-coded status, grade, grader, modified, and extension fields
into a preference, and add other grading fields:
 - gradevalue - actual grade value, separate from maximum grade.
 - grademax - maximum grade
 - created - submission creation date
 - latenessincext - how late the submission was, including any extensions

The fields gradevalue and grademax are intended as an alternative to
grade, suitable for further processing outside of Moodle, via
spreadsheet.

The default choices are set to the previous hard-coded fields.

NB: This has changed the field order in the columns.
Restrict name data available to only users with report/assign:shownames.
Moves the hard-coded files column into a preference, and adds columns
for whatever submission plugins are enabled.
Trying to track down which field is getting an extra column can be
pretty frustrating if there's no data that can be used to identify
the field - when it's just the '-' empty value placeholder.

Change false to true in the placeholder() function and it will put field
ids in for many (not all) placeholders.
Includes framework to adding more course fields easy.
@mjrox
Copy link

mjrox commented Feb 2, 2022

@thepurpleblob have you had an opportunity to review this pull request?

I'd like to pull from the main branch when it is merged.

@thepurpleblob
Copy link
Owner

I need to release a new version of this plugin which I'll try to get to in the next few days. I'll take a look at the same time.

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.

Allow selection of more/different assignment fields
3 participants