• last updated 2 hours ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Generate the fullscreen preview element in javascript and make it a child of the body, so that it can expand to the full page also when the include itself would be rendered in a smaller container

Implement a pre-selection filter for timeframes, which can be extended using downstream specific logics

    • -0
    • +7
    ./catalog/proctoring-support.de_DE.ISO-8859-1.xml
    • -0
    • +2
    ./catalog/proctoring-support.en_US.ISO-8859-1.xml
    • -0
    • +2
    ./catalog/proctoring-support.it_IT.ISO-8859-1.xml
    • -0
    • +21
    ./tcl/proctoring-callback-procs.tcl
Various changes on the proctoring artifacts users list:

- display some statistics on the collected artifacts for each user: count, number of reviewed, number of flagged, plus a colored progress bar indicating the review status, orange when in progress, red when completed with flags and green when everything is ok

- switched to a template::list to avoid too much handcrafting

- removed the red delete button from the top of the page, redundant, since we have the bulk actions on the list

- refined the behavior of the bulk-action delete button: it will be disabled when nothing is selected

- now we always trigger a reload when new artifacts are received for this object, but we cap it to 60 seconds

    • -0
    • +5
    ./catalog/proctoring-support.de_DE.ISO-8859-1.xml
    • -0
    • +5
    ./catalog/proctoring-support.en_US.ISO-8859-1.xml
    • -0
    • +5
    ./catalog/proctoring-support.it_IT.ISO-8859-1.xml
Store the string "false", not the boolean false

Use a full join so that in case of (normally minimal) unalignment between camera and desktop, we still display every artifact

Fix expression

Allow to filters also by timestamp, start or end

Provide an own translation for "All" that considers the specific item (e.g. for gendering)

    • -0
    • +1
    ./catalog/proctoring-support.de_DE.ISO-8859-1.xml
    • -0
    • +1
    ./catalog/proctoring-support.en_US.ISO-8859-1.xml
    • -1
    • +2
    ./catalog/proctoring-support.it_IT.ISO-8859-1.xml
Changes after review with downstream stakeholders:

- switch "Confirm OK" and "Flag" buttons

- put the two buttons after the comment section

Bonus change: mark unflagged artifacts with comments with a green border

Normalize the way the flag is represented in the JSON

Use a trash can icon, instead of all that yellow

Fix expression again

The flag coming from the ajax endpoint is an actual boolean

Make the position for the fullscreen preview sticky, and ensure the fullscreen element is positioned as the first body child, so it is put on top and can get as big as possible

Improve keyboard usability: focus the comment textarea when the modal opens and focus back on the comment button when the modal closes

Account for the scrollbar when enlarging the image

Enlarge images to fullscreen when one clicks on them

Don't set height explicitly

Fix filtering condition, improve doc

Significant improvements on the proctoring display UI to help the review process of proctored artifacts:

- user can flag artifacts as OK or to be reviewed

- user can add and delete comments on the single artifacts

- user can filter artifacts by "verified/not verified" "flagged/not flagged" and "all". A counter is displayed showing how many over the total number are being displayed

- interface will update as before when new pictures come in from an ongoing exam

- interface has been designed "mobile first" and with minimal, mostly graphical dependencies, to Bootstrap

    • -0
    • +8
    ./catalog/proctoring-support.de_DE.ISO-8859-1.xml
    • -0
    • +8
    ./catalog/proctoring-support.en_US.ISO-8859-1.xml
    • -0
    • +8
    ./catalog/proctoring-support.it_IT.ISO-8859-1.xml
file review.tcl was initially added on branch oacs-5-10.

Subst commands as well, include the artifact is in the websocket message

Formatting changes

Port the downstream only websocket enpoint

file proctoring-websocket.tcl was initially added on branch oacs-5-10.

    • -0
    • +0
    ./lib/proctoring-websocket.tcl
file websocket.tcl was initially added on branch oacs-5-10.

file websocket.adp was initially added on branch oacs-5-10.

Rename the file so it actually has an extension

Add extension to the filename so it is preserved

Improve upgrade:

- make output more visible when executed in the apm

- when atjobs are available, schedule the upgrade to run in the background at the next restart