Skip to content

fix: update ai summary and gap analysis once refresh is clicked#525

Merged
Roopan-Microsoft merged 1 commit intodevfrom
psl-refresh-button
Apr 9, 2026
Merged

fix: update ai summary and gap analysis once refresh is clicked#525
Roopan-Microsoft merged 1 commit intodevfrom
psl-refresh-button

Conversation

@Shreyas-Microsoft
Copy link
Copy Markdown
Collaborator

Purpose

This pull request introduces a mechanism to trigger refreshes across panels by adding a refreshTrigger counter to the Redux state and updating relevant components to react to its changes. The main goal is to ensure that when data is refreshed in the left panel, the center panel also re-fetches its data as needed.

State management enhancements:

  • Added a refreshTrigger numeric field to the LeftPanelState interface and initialized it in the Redux slice state (leftPanelSlice.ts). [1] [2]
  • Implemented the incrementRefreshTrigger action in the Redux slice to increment the refreshTrigger counter, and exported it for use in components. [1] [2]

Component updates for refresh handling:

  • In PanelLeft, after refreshing the grid, dispatches incrementRefreshTrigger() to signal that a refresh has occurred (PanelLeft.tsx). [1] [2]
  • In PanelCenter, subscribes to refreshTrigger from the Redux store and adds it as a dependency to the relevant useEffect hooks, ensuring that content and claim details are re-fetched when a refresh is triggered (PanelCenter.tsx). [1] [2]

Does this introduce a breaking change?

  • Yes
  • No

Golden Path Validation

  • I have tested the primary workflows (the "golden path") to ensure they function correctly without errors.

Deployment Validation

  • I have validated the deployment process successfully and all services are running as expected with this change.

What to Check

Verify that the following are valid

  • ...

Other Information

@Roopan-Microsoft Roopan-Microsoft merged commit 56ddc4c into dev Apr 9, 2026
7 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