Skip to content

Commit 7621380

Browse files
committed
Update task list UI and task categorization logic
Commented out in-progress step display in TaskList and simplified skeleton rendering logic. Modified TaskService to categorize completed tasks only by overall_status, removing PlanDataService.isPlanComplete check.
1 parent f1d3c44 commit 7621380

File tree

2 files changed

+7
-8
lines changed

2 files changed

+7
-8
lines changed

src/frontend/src/components/content/TaskList.tsx

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -49,9 +49,9 @@ const TaskList: React.FC<TaskListProps> = ({
4949
{task.date && task.status == "completed" && (
5050
<Caption1 className="task-list-task-date">{task.date}</Caption1>
5151
)}
52-
{task.status == "inprogress" && (
52+
{/* {task.status == "inprogress" && (
5353
<Caption1 className="task-list-task-date">{`${task?.completed_steps} of ${task?.total_steps} completed`}</Caption1>
54-
)}
54+
)} */}
5555
</div>
5656
<Menu>
5757
<MenuTrigger>
@@ -87,8 +87,8 @@ const TaskList: React.FC<TaskListProps> = ({
8787
<AccordionPanel>
8888
{loading
8989
? Array.from({ length: 5 }, (_, i) =>
90-
renderSkeleton(`in-progress-${i}`)
91-
)
90+
renderSkeleton(`in-progress-${i}`)
91+
)
9292
: inProgressTasks.map(renderTaskItem)}
9393
</AccordionPanel>
9494
</AccordionItem>
@@ -97,8 +97,8 @@ const TaskList: React.FC<TaskListProps> = ({
9797
<AccordionPanel>
9898
{loading
9999
? Array.from({ length: 5 }, (_, i) =>
100-
renderSkeleton(`completed-${i}`)
101-
)
100+
renderSkeleton(`completed-${i}`)
101+
)
102102
: completedTasks.map(renderTaskItem)}
103103
</AccordionPanel>
104104
</AccordionItem>

src/frontend/src/services/TaskService.tsx

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -39,8 +39,7 @@ export class TaskService {
3939

4040
// Categorize based on plan status and completion
4141
if (
42-
plan.overall_status === PlanStatus.COMPLETED ||
43-
PlanDataService.isPlanComplete(plan)
42+
plan.overall_status === PlanStatus.COMPLETED
4443
) {
4544
completed.push(task);
4645
} else {

0 commit comments

Comments
 (0)