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

Add option to allow bonus points in assessments #4090

Merged
merged 22 commits into from
May 29, 2021
Merged

Add option to allow bonus points in assessments #4090

merged 22 commits into from
May 29, 2021

Conversation

jonatanschroeder
Copy link
Member

@jonatanschroeder jonatanschroeder commented Apr 28, 2021

This option allows an assessment to have a score percentage over 100%.

  • Database, schema and sync;
  • Sprocs that update assessment score percentage;
  • Instructor view updates;
  • Student view updates;
  • API information about an assessment;
  • Docs;
  • Example course;
  • Tests.

@jonatanschroeder jonatanschroeder marked this pull request as ready for review April 29, 2021 04:25
@github-actions
Copy link
Contributor

github-actions bot commented May 6, 2021

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

@mwest1066 mwest1066 added this to In progress in Summer 2021 Development via automation May 24, 2021
@mwest1066 mwest1066 moved this from In progress to Review in progress in Summer 2021 Development May 24, 2021
Summer 2021 Development automation moved this from Review in progress to Done May 24, 2021
Summer 2021 Development automation moved this from Done to In progress May 24, 2021
@github-actions github-actions bot locked and limited conversation to collaborators May 24, 2021
@jonatanschroeder jonatanschroeder moved this from In progress to Review in progress in Summer 2021 Development May 24, 2021
@jonatanschroeder
Copy link
Member Author

Sorry about the moves up there, the GitHub app, when you open the details of a PR, gives you a Close button that I thought would close the details page, but it closes the PR instead without warning.

@PrairieLearn PrairieLearn unlocked this conversation May 28, 2021
Copy link
Contributor

@nwalters512 nwalters512 left a comment

Choose a reason for hiding this comment

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

Sorry for all the nits - grading is some of the most complex and potentially confusing stuff in PrairieLearn, so I want to make sure it's as clear and documented as possible! This looks good on the whole though.

I won't merge this quite yet in case Matt gets a chance to look later today - I'd trust his review of grading code more than my own 🙂

docs/assessment.md Outdated Show resolved Hide resolved
docs/assessment.md Outdated Show resolved Hide resolved
docs/assessment.md Outdated Show resolved Hide resolved
docs/assessment.md Outdated Show resolved Hide resolved
sprocs/assessment_instances_update.sql Outdated Show resolved Hide resolved
sprocs/assessment_instances_update.sql Outdated Show resolved Hide resolved
tests/testBonusPoints.js Outdated Show resolved Hide resolved
sprocs/assessment_instances_grade.sql Outdated Show resolved Hide resolved
Summer 2021 Development automation moved this from Review in progress to Reviewer approved May 28, 2021
Co-authored-by: Matthew West <matt@prairielearn.com>
@mwest1066 mwest1066 merged commit a941343 into master May 29, 2021
Summer 2021 Development automation moved this from Reviewer approved to Done May 29, 2021
@mwest1066 mwest1066 deleted the allow-bonus branch May 29, 2021 19:46
@github-actions github-actions bot locked and limited conversation to collaborators May 29, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

3 participants