Skip to content

PM-5203 approval phase ai only#1910

Open
vas3a wants to merge 6 commits into
devfrom
PM-5203_approval-phase-ai-only
Open

PM-5203 approval phase ai only#1910
vas3a wants to merge 6 commits into
devfrom
PM-5203_approval-phase-ai-only

Conversation

@vas3a
Copy link
Copy Markdown
Collaborator

@vas3a vas3a commented Jun 8, 2026

Related JIRA Ticket:

https://topcoder.atlassian.net/browse/

What's in this PR?

This pull request primarily refactors the AI Approval tab in the review app to remove the ability for admins and managers to directly edit and save workflow scores from the AI reviews table. Instead, users are now directed to view the full scorecard for a submission to inspect workflow scores. Additionally, the logic for generating response options for scorecard questions is consolidated into a utility function, reducing duplication across components.

Refactor and Removal of Inline Editing in AI Approval Tab:

  • Removed all state and logic related to inline editing, saving, and confirming workflow scores in the TabContentAiApproval component. Users can no longer edit or save scores directly from the AI reviews table. Instead, a "View scorecard" button is provided to navigate to the detailed scorecard view.
  • Updated UI text to reflect the new workflow ("Click View scorecard to inspect workflow scores.") and removed the edit/save/cancel buttons and confirmation modal.

Consolidation of Score Response Option Logic:

  • Replaced duplicated logic for generating score response options in both AppealComment and ManagerComment components with a single utility function getScoreResponseOptions, simplifying code and ensuring consistency.

These changes streamline the review workflow, reduce code duplication, and guide users to the appropriate place for managing workflow scores.

@vas3a vas3a requested review from jmgasper and kkartunov as code owners June 8, 2026 12:35
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