Skip to content

Allow reviewers to update task completion state from Review pane#2846

Draft
CollinBeczak wants to merge 1 commit into
mainfrom
Add-completion-state-update-ability-for-reviewers-MR3
Draft

Allow reviewers to update task completion state from Review pane#2846
CollinBeczak wants to merge 1 commit into
mainfrom
Add-completion-state-update-ability-for-reviewers-MR3

Conversation

@CollinBeczak

Copy link
Copy Markdown
Collaborator

Adds the Completion widget to the Review workspace defaults so reviewers can correct a task's completion status without rerouting through the mapper flow. The Completion widget renders ActiveTaskControls; when the new asReviewer prop is set, the controls:

  • dispatch the new updateTaskCompletionStatus action, which calls the existing review-status endpoint with the current reviewStatus preserved and newTaskStatus set, then stays on the current task
  • use the "for revision" allowed-status progressions so reviewers see valid status-change buttons even on Fixed/AlreadyFixed tasks
  • hide the mapper-only Revision Complete / next-task / needs-review / load-by controls (in both the widget and the confirmation modal)

Adds the Completion widget to the Review workspace defaults so reviewers
can correct a task's completion status without rerouting through the
mapper flow. The Completion widget renders ActiveTaskControls; when the
new asReviewer prop is set, the controls:

  - dispatch the new updateTaskCompletionStatus action, which calls the
    existing review-status endpoint with the current reviewStatus
    preserved and newTaskStatus set, then stays on the current task
  - use the "for revision" allowed-status progressions so reviewers see
    valid status-change buttons even on Fixed/AlreadyFixed tasks
  - hide the mapper-only Revision Complete / next-task / needs-review
    / load-by controls (in both the widget and the confirmation modal)

Meta-reviewers are unaffected — the action honors the existing
asMetaReview check from the URL.
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.

1 participant