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.

Sign up for GitHub

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

Jump to bottom

Activity completion support, improved scoring and logging #38

Closed
wants to merge 14 commits into from
Closed

Activity completion support, improved scoring and logging #38

wants to merge 14 commits into from

Conversation

sbourget
Copy link
Collaborator

This patch resolves issue #22 and also partially implements #14.

It addresses the following:

  • Adds tracking of user scores. Uses AJAX to write the scores to the database.
  • Improves logging allowing teachers / admins to identify who has been using the module.
  • Implements the callbacks for the outline and complete reports now that we have data that can be viewed on the reports.
  • Implements course reset as now we have scores that can be cleared.
  • Implements a page for teachers etc to view all student scores.
  • Adds support for activity completion for both views and minimum score
  • Adds unit tests for the events
  • Fixes remaining travis issues (travis is now passing)

@sbourget
mod_quizgame: Implement user scores
a689118 
This commit includes:
1. An AJAX service to write player scores to the DB
2. The ability to clear user scores upon activity deletion & course
reset.
3. User scores now display on the user activity reports (complete &
outline)
4. Improved logging for user scores.

TODO:
1. Activity completion based on score
2. Logging based on gameplay start
3. Unit Tests
4. Getting travis to pass.
@sbourget
mod_quizgame: Get travis to Pass
61a2a42 
Update travis.yml file to use v2 of the moodle-ci plugin
@sbourget
mod_quizgame: Add Activity completion support
0527140 
Adds support for both view and score based activity completion.
@sbourget
mod_quizgame: Small travis fixes
3089112
@sbourget
mod_quizgame: Log players when they start the game.
24e2610
@sbourget
mod_quizgame: Add additional fields to backup / restore
d8f5cc0
@sbourget
mod_quizgame: Small code checker fixes
78d5403
@sbourget
mod_quizgame: add quiz game generator & unit tests
f96cd14
@sbourget
mod_quizgame: Add unit tests for events.
1d27cca
@sbourget
mod_quizgame: Allow teachers to view player scores.
edc8a77
@sbourget
mod_quizgame: Small coding style fixes for travis
1927a57
@sbourget
mod_quizgame: Truncate scores before calling web service.
412b939 
With matching questions it is possible to get a score of 666.66667 with
will result in an exception since the database is expecting an integer.
@sbourget
mod_quizgame: fix typo in language string.
4e8ccdb
@sbourget
Copy link
Collaborator Author

sbourget commented Mar 2, 2018

@xow, When you have a chance, can you take a look at this patch

@sbourget sbourget closed this Mar 8, 2018
@sbourget sbourget deleted the ACTIVITY_COMPLETION branch March 8, 2018 14:59
@sbourget
Copy link
Collaborator Author

sbourget commented Mar 8, 2018

For some reason, github closed this pull request when I attempted to rebase one of the commits (fixing a whitespace issue) I've resubmitted the PR as #39

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@sbourget