Feature | Add note column to reviewer overview

[gabrielakuhn]

Summary
Add a new note field to the Review Overview table and display it as a column in the table.
Update the “Copy row” functionality to include the note value when copying row data.

Changes

• src/djehuty/web/database.py: Implement database procedure for update and delete a note using sentinel value and add check for RDF.nil in normalize biding.
• src/djehuty/web/formatter.py: Add note, submitter_account_uuid and reviewer_account_uuid to the response format
• src/djehuty/web/resources/html_templates/colors.css: Add global scope variables for gray colors
• src/djehuty/web/resources/html_templates/review/overview.html: Add note column to review table and styles
• src/djehuty/web/resources/sparql_templates/reviews.sparql: Add review note, submitter_account_uuid and reviewer_account_uuid to the return values for reviews
• src/djehuty/web/resources/sparql_templates/update_review.sparql: Modify sparql update template to include update of a note
• src/djehuty/web/resources/static/js/review-overview.js: Implement logic for adding, editing, and deleting notes and add note to copy row functionality
• src/djehuty/web/wsgi.py: implement update review note method

Approval Checklist

• I agree to follow Djehuty's code of conduct.
• I have read and I have follow the code contribution workflow.
• Code style and conventions were respected.
• Documentation has been updated where needed (README, docs, or examples).
• Review approved by at least one maintainer.
• Merge readiness (PR is squashed into a single commit and follows the commit template).

Issue Reference
We want to identify and implement new features in the deposit and publication workflow to make it easier and more efficient for users and data curators. Part of #43

Screenshots

[roelj]

Hi @gabrielakuhn! The PR looks very well done! I added an inline comment to the PR, and I'm looking forward to see if that idea of treating an empty string as an implicit delete of the note will work out. :-)

[gabrielakuhn]

Latest change: we now use a _UNSET as a sentinel value instead of rdf:nil when the note value is not provided.
rdf:nil means empty RDF list, so using it to signal “no value” was misleading. It’s also good RDF practice to do not have a triple at all when a value does not exist.