Skip to content

E2600 Reimplement review_mapping_controller: modify Review strategy tab#160

Open
z0brooks wants to merge 20 commits intoexpertiza:mainfrom
MATTMINWIN:main
Open

E2600 Reimplement review_mapping_controller: modify Review strategy tab#160
z0brooks wants to merge 20 commits intoexpertiza:mainfrom
MATTMINWIN:main

Conversation

@z0brooks
Copy link
Copy Markdown

Adding frontend support for review_mapping_controller within the review strategy tab (static and dynamic review strategy integration)

darrinj22 and others added 18 commits March 28, 2026 22:24
…ng_rendering

Djhanse2/review strategy mapping rendering
…gy_mapping_rendering"

This reverts commit ab6a63d, reversing
changes made to d448f71.
Revert "Merge pull request #15 from MATTMINWIN/djhanse2/review_strate…
…viewStrategy

Moved Review Strategy Mapping changes back into AssignmentEditor.tsx
…ent parameter payload values, ignoring myriad of 500 errors
Allow assignment updates to persist to backend by normalizing assignm…
…view-fields

added calibration specific tasks to static review mapping
set_allowed_number_of_reviews_per_reviewer: 0,
set_required_number_of_reviews_per_reviewer: 0,
static_reviewer_strategy: "",
static_assignment_sheet: File ? undefined : undefined,
Copy link
Copy Markdown

@johnmweisz johnmweisz Apr 2, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

File | undefined might be more clear

Zack Brooks and others added 2 commits April 3, 2026 15:55
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.

4 participants