Skip to content

Conversation

@Ravikirana-Microsoft
Copy link
Contributor

Purpose

This pull request includes multiple changes across the frontend and backend to improve functionality, enhance UI components, and optimize state management. The most significant updates include the addition of a reusable ContentNotFound component, replacement of custom accordion components with Fluent UI's accordion, and improvements to state handling for plan and task data.

Frontend Enhancements

1. Reusable ContentNotFound Component:

  • Added a new ContentNotFound component for displaying a user-friendly "not found" message, with customizable text, image, and buttons (src/frontend/src/components/NotFound/ContentNotFound.tsx).
  • Integrated ContentNotFound into PlanChat for scenarios where planData is unavailable (src/frontend/src/components/content/PlanChat.tsx). [1] [2]

2. Migration to Fluent UI Accordion:

  • Replaced custom Coral accordion components with Fluent UI's Accordion, AccordionItem, AccordionHeader, and AccordionPanel in TaskList for better consistency and maintainability (src/frontend/src/components/content/TaskList.tsx). [1] [2]

3. Enhanced Progress Circle:

  • Added a borderColor property to the ProgressCircle component and updated its rendering logic to include additional border styling (src/frontend/src/coral/components/Progress/ProgressCircle.tsx). [1] [2] [3]

State Management Improvements

4. Plan and Task Data Handling:

  • Introduced a new state variable allPlans in PlanPage to cache multiple plans and implemented logic to update planData based on the current planId (src/frontend/src/pages/PlanPage.tsx). [1] [2] [3]
  • Removed redundant local state updates in TaskDetails for step approval and rejection, delegating state updates to the main API functions (src/frontend/src/components/content/TaskDetails.tsx). [1] [2]

Backend Query Optimization

5. Query Simplification:

  • Modified the query in get_all_plans to remove the OFFSET 0 LIMIT 10 clause, ensuring all plans are retrieved without pagination (src/backend/context/cosmos_memory_kernel.py).

Does this introduce a breaking change?

  • Yes
  • No

How to Test

  • Get the code
git clone [repo-address]
cd [repo-name]
git checkout [branch-name]
npm install
  • Test the code

What to Check

Verify that the following are valid

  • ...

Other Information

@Ravikirana-Microsoft Ravikirana-Microsoft changed the base branch from main to dev July 4, 2025 07:02
@Roopan-Microsoft Roopan-Microsoft merged commit 5dff284 into dev Jul 4, 2025
7 checks passed
@Roopan-Microsoft Roopan-Microsoft deleted the Bugs-fixing branch July 4, 2025 07:11
@github-actions
Copy link

🎉 This PR is included in version 2.1.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

blessing-msft pushed a commit that referenced this pull request Aug 1, 2025
fix: Bugs fixing for 19597,19553,19524,19330,19278,19238
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants