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

Show Answer Details in Assessment Statistics Page (Part 1: Last Graded Answer (MCQ, MRQ, Text Response, File Upload)) #7130

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

Conversation

bivanalhar
Copy link
Contributor

@bivanalhar bivanalhar commented Feb 19, 2024

Background

This is the continuation of the previous PR #7077 and #7084 that adds feature allowing teacher to see the details of the answer when clicking on the grade box in the Assessment Statistics Page

This PR is related with the issue #4567

Feature Details

The display of the answer is as follows:
Screenshot 2024-02-20 at 7 43 17 AM

How to navigate:

  • In either Marks per Question Table or Attempt Count Table (available inside the Assessment Statistics Page), you might click on any grade/attempt-count box.
  • The answer details pop-up box will be shown immediately after.

Some consideration in the feature implementation:

  • Question is included in the Details as a reference for the teacher to make them easier to look into the question details while looking at the Answer.
  • Since our focus here is the details inside the answer, we made Question to be collapsible, with not expanded as the default.
  • We define the new component instead of redirecting the user to the Submission Page so that user can stay on the statistics page while examining the Answer

TO-DO Lists

Future Improvement

  • Define component for Voice Response and Scribing (at this point, not urgent to do)

Additional Notes

The codecov here noted some of the changes in BE not covered in the tests in RSpec. However, this will be later covered in the PR #7135

@bivanalhar bivanalhar force-pushed the bivan/teacher-stats-past-answer branch from dc5ec90 to 288e29c Compare February 19, 2024 23:54
@ekowidianto ekowidianto changed the base branch from master to bivan/teacher-statistics-attempt-count March 4, 2024 11:47
@bivanalhar bivanalhar force-pushed the bivan/teacher-statistics-attempt-count branch from 22881ce to d304444 Compare March 14, 2024 03:47
@bivanalhar bivanalhar force-pushed the bivan/teacher-stats-past-answer branch from 288e29c to 5829efd Compare March 14, 2024 03:49
@bivanalhar bivanalhar force-pushed the bivan/teacher-statistics-attempt-count branch 3 times, most recently from 0313c8d to a6cf1cd Compare April 2, 2024 06:36
Base automatically changed from bivan/teacher-statistics-attempt-count to master April 4, 2024 01:46
@bivanalhar bivanalhar force-pushed the bivan/teacher-stats-past-answer branch 2 times, most recently from 8aa52c6 to cd2abee Compare April 7, 2024 06:15
@bivanalhar bivanalhar force-pushed the bivan/teacher-stats-past-answer branch from cd2abee to f6a881f Compare July 24, 2024 03:39
- combine ancestor info and marks per question into main statistics
- remove all unnecessary controllers
- make the grade and attempt count box clickable
- each grade and attempt count cell is clickable
- after clicking, the box containing question will appear
- fetch question and answer details in one API
- use Accordion for question to allow for collapsing question
- include grade display inside answer box
- extend QuestionAnswerDetails to allow polymorphism
- reuse existing answer type def for defining its mini version
- Multiple Choice
- Multiple Response
- Text Response (with/without attachments)
- File Upload
- allowAttachment to be changed into maxAttachments nonzero
- fix rubocop trailing whitespace issue
@bivanalhar bivanalhar force-pushed the bivan/teacher-stats-past-answer branch from f6a881f to a4b00fc Compare July 24, 2024 04:11
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.

None yet

1 participant