Skip to content

AB#32012 fix review list AI scoring button gating#2168

Merged
jacobwillsmith merged 1 commit into
devfrom
feature/AB#32012-RefactorAIFlowOrchestration
Mar 25, 2026
Merged

AB#32012 fix review list AI scoring button gating#2168
jacobwillsmith merged 1 commit into
devfrom
feature/AB#32012-RefactorAIFlowOrchestration

Conversation

@jacobwillsmith
Copy link
Copy Markdown
Contributor

Pull request overview

This pull request fixes the AI Generate and Clone button gating in the Review List component. The component-owned hidden flag for AI scoring availability was rendering in a way that the browser exposed as "value" instead of an explicit boolean string, which caused the Review List JavaScript to treat AI scoring as disabled even when the page-level scoring gate was enabled.

Changes:

  • Fixed the Review List hidden input to render an explicit True or False string for AI scoring availability
  • Restored the Review List AI Generate and Clone button visibility logic to respect the component-owned scoring gate correctly
  • Kept the Review List gating model aligned with the existing feature-plus-permission checks for AI scoring

@github-actions
Copy link
Copy Markdown

🧪 Unit Test Results (Parallel Execution)

Tests

📊 Summary

Result Count
✅ Passed 561
❌ Failed 0
⚠️ Skipped 0

📄 HTML Reports

  • Merged Tests (HTML): Included in artifacts
    Generated automatically by CI.

@jacobwillsmith jacobwillsmith force-pushed the feature/AB#32012-RefactorAIFlowOrchestration branch from 51657ef to 858b034 Compare March 25, 2026 22:23
@github-actions
Copy link
Copy Markdown

🧪 Unit Test Results (Parallel Execution)

Tests

📊 Summary

Result Count
✅ Passed 561
❌ Failed 0
⚠️ Skipped 0

📄 HTML Reports

  • Merged Tests (HTML): Included in artifacts
    Generated automatically by CI.

@jacobwillsmith jacobwillsmith marked this pull request as ready for review March 25, 2026 22:29
@jacobwillsmith jacobwillsmith merged commit 2326170 into dev Mar 25, 2026
22 checks passed
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.

2 participants