-
Notifications
You must be signed in to change notification settings - Fork 2
Feat grade report #66
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
Open
ae324
wants to merge
116
commits into
master
Choose a base branch
from
feat-grade-report
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
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Bug fixes needing resolution: tasks with completely blank fields (e.g. "2": {
"type": "-",
"name": "-",
"value": "-",
"max": "-",
"weight": "-",
"convertedNumericValue": "-",
"scaledGrade": "-"
},
) cause an error in rendering the MultiTaskGradeTable
…file doesn't automatically download)
This was done in order to call GET /sectionUserInfo/:userID/:sectionID, so that a user could be identified as a student or instructor and thereby call POST studentGradeReport or gradeReport accordingly.
…rade report as necessary.
…for download, and styled table subheading
…assignment selected to be used for the filename of the exported version of the AGR.
These updates are made from changes proposed in the Grade_Report_Fixes_2022_01-25 document. - Separate student/instructor grade report APIs developed - Grade report tables now include student/course/semester/section identifying information to better orient users - Tooltips across (mostly) all grade report tables (i.e. essentially within each column) - Bypassing of certain reports and/or removal of certain columns when no relevant information is present (e.g. bypassing of Problem Grade Report under only one workflow, removal of extra credit columns in AGR when there are no extra credit tasks) - Calculation of totals under (mostly) each table to help students better understand how grades in the "upper" tables are derived from the "lower/drill-down" tables - Additional minor updates
f466efa to
5ed472a
Compare
- fix error of PTTR not appearing when oneWorkflow=true - used APIs recently implemented in the backend to show courses and sections relevant (i.e. more specific) to the student
- Instructions are now at the top of the report on how to widen the tables/columns - Upon clicking drill-down links, the page automatically scrolls to the newly loaded tables!
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
No description provided.