Skip to content

f-1563 improve the display of questions in the manual grading interface - #1581

Open
morozov-av wants to merge 4 commits into
RunestoneInteractive:mainfrom
morozov-av:f-1563
Open

morozov-av wants to merge 4 commits into
RunestoneInteractive:mainfrom
morozov-av:f-1563

Conversation

@morozov-av

Copy link
Copy Markdown
Contributor
  1. Convert multiple-choice indexes to option letters (0 → A, 1 → B)
  2. Reconstruct and display Parsons block order
  3. Display Matching and Drag-and-Drop answers as readable labeled pairs instead of raw JSON
  4. Preserve saved interactive attempts and required Drag-and-Drop sizing metadata
  5. Refactor questionTypes into colocated component folders with dedicated utilities, barrel exports, and tests
f-1563.mp4
image

@morozov-av
morozov-av requested a review from bnmnetp as a code owner October 2, 2026 18:21
@bnmnetp

bnmnetp commented Oct 5, 2026

Copy link
Copy Markdown
Member

I'm not seeing anything different in this view:

Screenshot 2026-10-05 at 2 55 12 PM

I think when most instructors want to manually grade they use this. Many have requested this view where they select all questions and then just one student so they can see all of the students work for a single assignment.

@morozov-av

Copy link
Copy Markdown
Contributor Author

I'm not seeing anything different in this view:

Screenshot 2026-10-05 at 2 55 12 PM I think when most instructors want to manually grade they use this. Many have requested this view where they select all questions and then just one student so they can see all of the students work for a single assignment.

Thanks. I’ve updated manually grad dialog

f-1563_1.mp4

@bnmnetp

bnmnetp commented Oct 5, 2026

Copy link
Copy Markdown
Member

Great progress. Ideally the answer for the parsons problems would just look like the code in plain text as the student arranged it. that would work for other parsons questions as well as the coding problems.

See: https://docs.runestone.academy/en/latest/parsons_answer_format.html

@morozov-av

Copy link
Copy Markdown
Contributor Author

@bnmnetp
Tests were failing due to changes introduced in c8b7298
Profile email test used a different async event loop, causing cascading asyncpg failures, and the email form field remained required despite the new blank-email scenario

@bnmnetp

bnmnetp commented Oct 5, 2026

Copy link
Copy Markdown
Member

Thanks, I run the tests locally with every build.... I must have missed something when I was merging that PR. I'm looking into it now.

@bnmnetp

bnmnetp commented Oct 5, 2026

Copy link
Copy Markdown
Member

OK, fixed it up here: 7cf6ad9

@ascholerChemeketa -- this should still do what you wanted.

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.

2 participants